- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] use LOG_STR_LEN for error/warning/debug messages, too
by VMiklos 06 May '06

06 May '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; the current buffer's length is only 9 chars which is ok for English but not for other languages udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

1 0

[pacman-dev] [PATCH] makepkg: handle spaces in source filenames
by VMiklos 28 Apr '06

28 Apr '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; i know, it does not apply cleanly to the cvs, but Aurel already said previously it's not a problem, so then i won't fix it up udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

1 2

[pacman-dev] [PATCH] libalpm: yet another fix for conflits&provides
by VMiklos 10 Apr '06

10 Apr '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; the scenario is the following: pkg B provides and conflicts with pkg A pkg E depends on pkg C and pkg D pkg C depends on pkg A and pkg D depends on pkg B so during a -S E, pkg A should be removed from the target list no log message needed we already have: debug: package 'xespgs' provides its own conflict flow2: removing 'espgs' from target list which should be enough imho applies to the latest Arch cvs maybe it would be good to create a test for this scenario udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 3

[pacman-dev] [PATCH] include config->root in conflict error messages
by VMiklos 31 Mar '06

31 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

1 0

[pacman-dev] [patch] 2.X series: add scriptlet output to pacman's logfile
by Aaron Griffin 31 Mar '06

31 Mar '06

With all the hoo-hah with people missing scriptlet messages, I figured this would be a nice addition. It adds a dependancy on 'tee', so it may just fail outright is coreutils is uninstalled, but if you're uninstalling coreutils, you deserve it.

1 0

[pacman-dev] [PATCH] pmdb_t: allow repo names to be longer than 128 chars
by VMiklos 22 Mar '06

22 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; any reason for limiting the length of the repo names? i would suggest increase the limit to PATH_MAX udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 2

[pacman-dev] cvs problems
by VMiklos 22 Mar '06

22 Mar '06

hi $ cvs update cvs [update aborted]: connect to cvs.archlinux.org(66.211.213.17):2401 failed: Connection refused is this a known problem? udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 2

[pacman-dev] [PATCH] pacman: query dbpath before using it
by VMiklos 21 Mar '06

21 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; fix for the ugly "DBPath: (null)" message when running pacman -v udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] [PATCH] alpm_db_update(): remove duplicated db->treename from path
by VMiklos 21 Mar '06

21 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; s|/var/lib/pacman/mypkgs/mypkgs|/var/lib/pacman/mypkgs| udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] [PATCH] _alpm_db_read(): fix install file's location
by VMiklos 21 Mar '06

21 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; so that pre/post_remove will work again udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] libalpm and backends
by Aurelien Foret 15 Mar '06

15 Mar '06

Hello, As a first step toward backend support, I slightly tweaked the database code from libalpm into this direction. Basically, I've moved all functions relating to the backend from db.c inside a new file "be_files.c". The idea is to have several distinct "be_XXX.c" files, one for each backend, implementing the hooks to access database fields. The functions that need to be implemented in order to write a new backend are: _alpm_db_open _alpm_db_close _alpm_db_rewind _alpm_db_scan _alpm_db_read _alpm_db_write _alpm_db_remove So, the next step should be to allow users to choose a backend with the configure script as follows ("files" is the default behavior): ./configure --backend=files or ./configure --backend=gdbm This option should impact the list of targets built in the libalpm library: if gdbm is activated, only the object "be_gdbm.c" should be compiled and linked inside the library. As of now, what is missing is to implement this selection in the configure.ac script and in the lib/libalpm/Makefile.am file. Currently, the Makefile.am is as follows: TARGETS = packege.o db.o cache.o list.o ... TARGETS += be_files.o This last line should be added conditionally depending on configure options. And the final big step is to revamp the sync db archives handling. To date, the database synchronization can't be easier: the library does not even need to read the archive, it simply unpacks it in /var/lib/pacman/<treename>. It works well because gensync archives are mapped on the current "flat files" database format. Now, let's imagine we're using a different backend... that can't work anymore: the library must read the archive, parse each entry and db_write it to the database. The following patch is implementing the whole thing: http://aurelien.foret.free.fr/archlinux/pacman/patches/pacman-lib-backend.p… Although the modifications are quite small and straightforward, I'm submitting this patch for review to the pacman-dev ml because it has an obvious external impact on sync db archives. Here are some explanations. To ease the pain, the idea is to rework gensync to simplify the sync db archive content. I reused the format from .PKGINFO files as used by makepkg and implemented it in gensync. For instance, in these conditions, an entry from the sync db archive is a file named "dummy-1.0-1" with the following content: pkgname = dummy pkgver = 1.0-1 depend = foobar1 replaces = foobar2 csize = 123456789 ... instead of a subdirectory "dummy-1.0-1" containing two files: dummy-1.0-1/desc: %NAME% dummy %VERSION% 1.0-1 ... and dummy-1.0-1/depends: %DEPENDS% foobar1 ... This way, it is possible to reuse the function parsedesc from package.c to parse both .PKGINFO files from package archives _and_ entries from database archives. IMO, this is quite efficient and simple: makepkg and gensync are using the same format, and gensync database archives are simpler to parse and analyze. Finally, I think pacman 3.0 is the right moment to implement it. We already agreed to break compatibility for sync databases by moving "replaces" and "force" fields from "desc" to "depends" db files, and as a consequence pacman 3 release will already require users to synchronize their sync databases before being able to use them. The gensync rework I'm suggesting will don't bring more hassle to the upgrade from pacman 2.9 to pacman 3.0. I'm interested to get feedbacks on the general idea and the implementation, before possibly committing it. -- Aurelien

2 3

[pacman-dev] [PATCH] fix the install file's location
by VMiklos 14 Mar '06

14 Mar '06

hi http://frugalware.org/~vmiklos/patches/libpacman-proposed/install_location.… ie. the location of the scriptlet of the pacman-tools package should be /var/lib/pacman/local/pacman-tools-0.6.8-1/install not /var/lib/pacman/pacman-tools-0.6.8-1/install udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] pactest 0.3
by Aurelien Foret 11 Mar '06

11 Mar '06

I've released a new version of the test suite. I've set up a small web page about the project: http://aurelien.foret.free.fr/archlinux/pactest/ This way I'll stop polluting the pacman-dev mailing list. Or may I add a "test" subdirectory in the CVS repository to put the testsuite in it? ;) -- Aurelien

3 3

[pacman-dev] [PATCH] --noscriptlet & PM_TRANS_FLAG_NOSCRIPTLET
by VMiklos 07 Mar '06

07 Mar '06

hi if you install packages in a chroot just as a dependency and remove them just after the build, then it worth to skip the pre/post_install/remove scriptlets http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; this patch implements this for libalpm & pacman yes, the patch does not apply cleanly, there is a failed hunk, but Aurelien said before that's the a big problem so i won't fix it also probably the the value of PM_TRANS_FLAG_NOSCRIPTLET should be changed to 0x100 udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] TODO update
by Aurelien Foret 05 Mar '06

05 Mar '06

Hello, I've closed items: [3] code synchro with pacman 2.9.8 [5a] db_write: database entries cleanup [11] _alpm prefix to private library functions On-going items: [7] (vmiklos) [8] (aurelien) the .lastupdate code was moved from the library to the frontend. FYI, I'm currently working on item [6] Here is the updated TODO: [2] possible db corruption because of pmsyncpkg_t struct: the spkg field is just a pointer to data from the cache, which is likely to change during sync_commit [5] database cleanup b) do not add name and version entries in "desc" file (already available in the directory name), and also for .PKGINFO file (in makepkg)? => on hold for now [6] more debug messages in libalpm some parts of the code still remain obscure (file conflicts handling for instance) even when running pacman with "--debug=-1" [7] remove exit() calls from libalpm the library should never call exit() but always return an error code to the frontend (for instance, drop MALLOC macro) [8] .lastupdate review Is the .lastupdate file really needed? Using mtime isn't enough? [9] review file conflict code See this thread: http://www.archlinux.org/pipermail/pacman-dev/2006-February/000202.html [10] pacman -Sw Outputs are wrong compared to pacman 2.9 Do not use log callbacks when the -w is used? or filter all outputs in trans.c depending on the downloadonly flag? => it should be done as the last step before a first release to avoid introducing too much changes to the CVS code at once. -- Aurelien

2 6

[pacman-dev] improving --ignore
by VMiklos 02 Mar '06

02 Mar '06

hi what about to the following improvement: old: $ sudo pacman -S misc-fonts --ignore corefonts :: group misc-fonts: corefonts fontconfig fontforge :: Install whole content? [Y/n] :: corefonts-1.3r4-4: local version is up to date. Upgrade anyway? [Y/n] :: fontforge-20060125-1: local version is up to date. Upgrade anyway? [Y/n] resolving dependencies... done. looking for inter-conflicts... done. Targets: corefonts-1.3r4-4 fontconfig-2.3.93-1 fontforge-20060125-1 Total Package Size: 6.8 MB Proceed with upgrade? [Y/n] n new: $ sudo pacman -S misc-fonts --ignore corefonts :: group misc-fonts: corefonts fontconfig fontforge :: Install whole content? [Y/n] :: About to install corefonts, but it is in IgnorePkg. Install anyway? [Y/n] :: corefonts-1.3r4-4: local version is up to date. Upgrade anyway? [Y/n] :: fontforge-20060125-1: local version is up to date. Upgrade anyway? [Y/n] resolving dependencies... done. looking for inter-conflicts... done. Targets: corefonts-1.3r4-4 fontconfig-2.3.93-1 fontforge-20060125-1 Total Package Size: 6.8 MB Proceed with upgrade? [Y/n] n comments? udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 2

[pacman-dev] release roadmap?
by VMiklos 01 Mar '06

01 Mar '06

hi all just wanted to suggest let's make a list of bugs that blocks us to release a public rc (and then the first stable) release of pacman3 we have several ideas, for example replacing the ugly chroot system() call, adding gettext support, etc, but these features were not in pacman2, and i think they're not "musthave" so what are the points from TODO that must be solved before a release? i think the only one is the lack of handling of package conflicts during packages replacement, and maybe the manpages of public funtions (via doxygen) udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

3 37

[pacman-dev] [PATCH] pacman: disable progressbar by default if the output is redirected
by VMiklos 01 Mar '06

01 Mar '06

hi http://darcs.frugalware.org/darcsweb/darcsweb.cgi?r=pacman;a=plain_commitdi…; idea from wget ;) udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 1

[pacman-dev] [PATCH] remove corrupted packages if requested
by VMiklos 28 Feb '06

28 Feb '06

hi http://frugalware.org/~vmiklos/patches/libpacman-proposed/unlink_corrupt.di… udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

1 0

[pacman-dev] [PATCH] use sudo everywhere for makepkg -iS
by VMiklos 28 Feb '06

28 Feb '06

hi http://frugalware.org/~vmiklos/patches/libpacman-proposed/makepkg_install.d… if a package already built, and makepkg was called with -iS, then we should use sudo to install the package. if there were no -S and we're not root, then we should inform the user about we can't auto-install the packages - the same way as we do with the dependency auto-install. as the package install code was duplicated, i've moved it to a separate function udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

1 0

[pacman-dev] Away
by Aurelien Foret 24 Feb '06

24 Feb '06

I'll be away for the next 3 or 4 days (skiing). I hope the patch queue will not inflate too much in the meantime ;) -- Aurelien

1 0

[pacman-dev] [PATCH] whitespace fixes for libalpm
by VMiklos 24 Feb '06

24 Feb '06

hi http://frugalware.org/~vmiklos/patches/libpacman-proposed/alpm_db_cosmetics… somehow these tabs are replaced with spaces udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

3 5

[pacman-dev] test/sync999 - pacman segfault
by Aurelien Foret 23 Feb '06

23 Feb '06

Hi, Running pactest today showed that sync999 test case no longer works. Eventually, pactest return on investment does not seem null anymore :) It seems the flaw appeared with lib/libalpm/conflict.c:1.15. The same test with conflict.c:1.14 is ok. Here's the log from pactest: flow2: PM_OPT_DBPATH set to 'var/lib/pacman' flow2: PM_OPT_CACHEDIR set to 'var/cache/pacman/pkg' debug: opening database 'local' debug: opening database 'sync' :: Starting local database upgrade... flow1: checking for package replacements debug: loading package cache (infolevel=0x3) for repository 'sync' flow1: checking for package upgrades debug: loading package cache (infolevel=0x3) for repository 'local' debug: CACHE: fetching 'sync/pkg1' flow2: pkg1-1.0-1 elected for upgrade (1.0-1 => 1.0-2) debug: CACHE: fetching 'sync/pkg2' flow2: pkg2-1.0-1 elected for upgrade (1.0-1 => 1.0-2) resolving dependencies... flow1: resolving targets dependencies done. looking for inter-conflicts... flow1: looking for unresolvable dependencies debug: CACHE: fetching 'local/pkg1' debug: CACHE: fetching 'local/pkg2' flow1: looking for conflicts debug: targs vs db: found pkg2 as a conflict for pkg1 debug: targs vs targs: found pkg2 as a conflict for pkg1 debug: db vs targs: found pkg1 as a conflict for pkg2 flow2: package 'pkg1' is conflicting with 'pkg2' debug: CACHE: fetching 'local/pkg2' debug: package 'pkg1' provides its own conflict debug: resolving package 'pkg1' conflict flow2: electing 'pkg2' for removal flow2: removing 'pkg2' from target list flow2: package 'pkg2' is conflicting with 'pkg1' debug: CACHE: fetching 'local/pkg1' When handling conflict pkg1, pkg2 is removed from targets. Then, the library tries to handle pkg2 conflict... but find_pkginsync() returns NULL... I've committed a fix (sync.c:1.66) to skip conflict resolution in such a case. Thoughts? The test suite is ok with it ;) -- Aurelien

2 4

[pacman-dev] Sorry, test mail for me :)
by Christian Hamar 23 Feb '06

23 Feb '06

Hey. Just a test email, sorry from list members, i need to figure out why i got bounced on this list :( Regards -krix-

1 0

[pacman-dev] makepkg: removing empty directories?
by VMiklos 21 Feb '06

21 Feb '06

from makepkg: 1.12 (aurelien 16-Feb-06): # remove empty directories 1.12 (aurelien 16-Feb-06): if [ "`check_option NOEMPTYDIRS`" ]; then 1.12 (aurelien 16-Feb-06): msg "Removing empty directories..." 1.12 (aurelien 16-Feb-06): find pkg -mindepth 1 -type d -empty -exec rmdir {} \; 1.12 (aurelien 16-Feb-06): fi cmiiw, but openssh creates an empty /var/empty directory and it _does_ need it. any idea how to solve this problem? for now, i haven't merged this hunk to our tree udv / greetings, VMiklos -- Developer of Frugalware Linux, to make things frugal - http://frugalware.org

2 2