- Pacman-dev - lists.archlinux.org

[pacman-dev] How to run the install scripts
by Siem Korteweg 10 Jul '07

10 Jul '07

Hi, hope this is the proper mailing-list to post this. I am new to ArchLinux and developing a package. I am having trouble to get the install scripts running, I tested the following (minimal) package: file x.install: echo "$0 with arguments: $*" >>/tmp/log # to be extended for pre- and post-install actions exit 0 file src/x #!/bin/sh echo "$0" exit 0 file x.build: pkgname=x pkgver=1.1.1 pkgrel=1 pkgdesc=('get the install scripts running') arch=('i686') install=x.install license=('GPL') build(){ mkdir -p ../pkg/opt/x/bin cp -x ../pkg/opt/x/bin chmod 700 ../pkg/opt/x/bin/x } building the package with the command: makepkg -p x.build The build succeeds and the resulting x-1.1.1-1.i686.pkg.tar.gz file contains all the right files and the .INSTALL file contains the code of the x.install file. Installing the package with: pacman -U x-1.1.1-1.i686.pkg.tar.gz succeeds also. The file /opt/x/bin/x is created. However, the file /tmp/log does not exist, while I expected it to contain several lines for at least pre-install and post-install. I did not change any of the pacman configuration files (on my base-install). Why does the install script not run? Thanks in advance for any clue. Siem Korteweg

3 4

[pacman-dev] CVS update of pacman-lib (configure.ac)
by Dan McGee 10 Jul '07

10 Jul '07

Date: Tuesday, July 10, 2007 @ 11:08:33 Author: dan Path: /home/cvs-pacman/pacman-lib Modified: configure.ac (1.42 -> 1.43) backport fix for FS 7578 --------------+ configure.ac | 3 +++ 1 file changed, 3 insertions(+) Index: pacman-lib/configure.ac diff -u pacman-lib/configure.ac:1.42 pacman-lib/configure.ac:1.43 --- pacman-lib/configure.ac:1.42 Sun Jun 17 20:15:36 2007 +++ pacman-lib/configure.ac Tue Jul 10 11:08:33 2007 @@ -148,6 +148,9 @@ AC_MSG_ERROR("libdownload is needed to compile pacman!"); fi +# Enable large file support if available +AC_SYS_LARGEFILE + dnl Set config location AC_MSG_CHECKING(for configuration file name) if test -n "$configfile"; then

1 0

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-272-g4906e15
by Dan McGee 09 Jul '07

09 Jul '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via 4906e15d0d31bf0442c25af44bcb637b87a7b027 (commit) via a2186a43a895950cf97dcfcec4b18279f09f0801 (commit) from a501b72e40062c9e606a787657582d1859c35d35 (commit) - Log ----------------------------------------------------------------- commit 4906e15d0d31bf0442c25af44bcb637b87a7b027 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jul 9 00:46:29 2007 -0400 Remove gettext from any alpm_logaction calls We shouldn't translate log messages to pacman.log so it is consistant and can be parsed by other tools. Remove all gettext _() around these strings. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a2186a43a895950cf97dcfcec4b18279f09f0801 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Jul 8 23:46:48 2007 -0400 Add top level hidden file code to package.c This was already taken care of in add.c by commit 2ae043866045bbeef129a4b503ee1f5486545821, but was missed in package.c. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: lib/libalpm/add.c | 38 +++++++++++++++++++++++--------------- lib/libalpm/package.c | 3 +++ lib/libalpm/sync.c | 4 ++-- lib/libalpm/trans.c | 2 +- src/pacman/callback.c | 6 +++--- src/pacman/sync.c | 4 ++-- 6 files changed, 34 insertions(+), 23 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ a501b72e40062c9e606a787657582d1859c35d35 \ 4906e15d0d31bf0442c25af44bcb637b87a7b027

1 0

[pacman-dev] little error when installing/upgrading the first package in a new database
by Xavier 09 Jul '07

09 Jul '07

There was an odd error when the first package installed in a database had a scriptlet : LANG=C sudo src/pacman/pacman --conf pacman.conf -Ud /var/cache/pacman/pkg/filesystem-0.8-3.pkg.tar.gz loading package data... done. (1/1) checking for file conflicts [####################################################################################################] 100% error: could not open file /home/xav/dev/pacman/foo/var/lib/pacman/local//filesystem-0.8-3/depends: No such file or directory----------------------------] 0% (1/1) installing filesystem [####################################################################################################] 100% This doesn't happen with package that doesn't have a scriptlet, and it doesn't happen either when there is at least one other package in the database. But it still happened after running the same pacman command several times (ie upgrading the single package in the database). I finally found why. in add.c, alpm_add_commit function : 366 _alpm_log(PM_LOG_DEBUG, _("removing old package first (%s-%s)"), oldpkg->name, oldpkg->version); That part will remove the filesystem package if it existed (in case of an upgrade), and will also remove the entry from the database (filesystem directory in $dbpath/local) 473 } else if(strcmp(entryname, ".INSTALL") == 0) { 474 /* the install script goes inside the db */ 475 snprintf(filename, PATH_MAX, "%s/%s-%s/install", db->path, 476 newpkg->name, newpkg->version); That extracts the scriptlet to $dbpath/local//filesystem-0.8-3/install . then later, we have this : 778 /* Update the requiredby field by scanning the whole database 779 * looking for packages depending on the package to add */ 780 _alpm_pkg_update_requiredby(newpkg); and later, this : 821 _alpm_log(PM_LOG_DEBUG, _("updating database")); 822 _alpm_log(PM_LOG_DEBUG, _("adding database entry '%s'"), newpkg->name); So when update_requiredby is called, the entry hasn't been totally written in the database, we only have the install file, but no depends or desc file. and some code from update_requiredby in package.c : 756 pmdb_t *localdb = alpm_option_get_localdb(); 757 for(i = _alpm_db_get_pkgcache(localdb); i; i = i->next) { ... 764 for(j = alpm_pkg_get_depends(cachepkg); j; j = j->next) { So _alpm_get_pkgcache will return the filesystem package, because the directory exists in $dbpath/local/ , but alpm_pkg_get_depends will try to read the depends file which doesn't exist yet, resulting in the error message. Finding this didn't take too long, but this error should then appear in every cases, as long as the package has a scriptlet. But it didn't, as soon as there were more than one package in the db, the error disappeared. That's because of the pkg cache. in package.c , _alpm_db_get_pkgcache function, we have : 106 if(!db->pkgcache) { 107 _alpm_db_load_pkgcache(db); 108 } so the package database is loaded again only if db->pkgcache is NULL. The first time we install filesystem, pkgcache will be NULL when that function is called, so it'll read the database, finding the partial filesystem entry. And when we upgrade filesystem, it removes the old filesystem package first and at this moment _alpm_db_remove_pkgfromcache function from cache.c will be called, doing this : 152 _alpm_log(PM_LOG_DEBUG, _("removing entry '%s' from '%s' cache"), 153 alpm_pkg_get_name(pkg), db->treename); 154 155 db->pkgcache = alpm_list_remove(db->pkgcache, pkg, _alpm_pkg_cmp, &vdata); which will result in a NULL db->pkgcache too. There is no differences between an uninitialized list and an empty list, unfortunately ... When there is another package in the database, pkgcache won't be NULL after the current package being upgraded is removed from pkgcache list. So the pkgcache won't be updated by reading the database again when update_requiredby is called, and so it won't fail on a broken partial filesystem entry, because it has been removed from pkgcache ealier. While that issue was totally harmless, and never happens, because there is always more than one package in the db, it was annoying to get it every time when debugging, without knowing where it came from. For fixing it, it shouldn't be too hard, but I'm not sure yet. Maybe we can delay the extraction of the scriptlet to $dbpath/ , and only installs it in the same time than the dabase entry is created. Or maybe we can just call requiredby after the database entry is fully created, I'm not sure.. Or maybe we can make the difference between an empty pkgcache and a not initialized one.

3 5

Re: [pacman-dev] FS#7525: Permissions of .pacnew files
by Andrew Fyfe 08 Jul '07

08 Jul '07

Move the discussion to the ML... Pacman wrote: > THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. > > The following task has a new comment added: > > FS#7525 - Permissions of .pacnew files > User who did this - Xavier (shining) > > ---------- > Well then, maybe we could read the whole archive a first time, for > determining how to handle backup files (on the fly md5 computation), > and we record this information somehow. > And then a second time, where we actually extract the files. > > Otherwise, the files in backup array are always small config files > afaik, but well, it could be indeed be safer > to avoid putting the whole file in memory. > > Oh well, finally, the current behavior is alright I guess, and this is > probably not the correct place for discussing about things anyway (silly > me). > So I think this bug could be closed now :) > ---------- I've been thinking of another solution, include some more information in .FILELIST eg: # directory name d usr d usr/bin # file name checksum f usr/bin/foo md5sum # link name target l usr/bin/bar foo This way we can get the md5sum from the file list rather than having to extract the file to get it's md5sum. Andrew

2 2

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-270-ga501b72
by Dan McGee 07 Jul '07

07 Jul '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via a501b72e40062c9e606a787657582d1859c35d35 (commit) via b5f8a44bebc906bf6a29d30c159802b0c1a7dbb1 (commit) via 49f447d02c803e5a2f63582ce78cabb850ebfa89 (commit) via 15e1ce2e709e0a16dd54ea3b5eaab0003e32b62d (commit) from 6b7b9743181078aa7152daffdfc1eaeb46304c0f (commit) - Log ----------------------------------------------------------------- commit a501b72e40062c9e606a787657582d1859c35d35 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Jul 6 19:35:32 2007 -0400 Make paragraph text in scripts single strings It is much easier for translators to deal with paragraphs as strings rather than by lines. Take all usage and version information and convert it to this format. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b5f8a44bebc906bf6a29d30c159802b0c1a7dbb1 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Jul 6 18:43:24 2007 -0400 Move scripts from *.in to *.sh.in so gettext can determine type If we move the scripts from *.in to *.sh.in and *.py.in, gettext can pull the required strings to translate a whole lot easier. Do this. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 49f447d02c803e5a2f63582ce78cabb850ebfa89 Author: Dan McGee <dan(a)archlinux.org> Date: Fri Jul 6 17:59:08 2007 -0400 First step of moving translations from src/pacman/po to po/ Move the translations from src/pacman/po to just po/ so we can include the scripts gettext translations in the same message catalog as that of the pacman frontend. The libalpm message catalog, for now, will remain a separate existence. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 15e1ce2e709e0a16dd54ea3b5eaab0003e32b62d Author: Dan McGee <dan(a)archlinux.org> Date: Fri Jul 6 12:11:55 2007 -0400 Various small fixes as suggested by some static code checkers I ran flawfinder and sparse over the pacman source code and found a few things that were worth fixing (and were quick fixes). Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: Makefile.am | 4 +- configure.ac | 2 +- lib/libalpm/add.c | 7 - lib/libalpm/alpm.c | 5 +- lib/libalpm/alpm.h | 2 +- lib/libalpm/util.c | 8 +- {lib/libalpm/po => po}/.gitignore | 0 {lib/libalpm/po => po}/LINGUAS | 0 {lib/libalpm/po => po}/Makefile.in.in | 0 {src/pacman/po => po}/Makevars | 2 +- po/POTFILES.in | 22 ++ {src/pacman/po => po}/de.po | 0 {src/pacman/po => po}/en_GB.po | 0 {src/pacman/po => po}/es.po | 0 {src/pacman/po => po}/fr.po | 0 {src/pacman/po => po}/hu.po | 0 {src/pacman/po => po}/it.po | 0 {src/pacman/po => po}/pacman.pot | 0 {src/pacman/po => po}/pl_PL.po | 0 {src/pacman/po => po}/pt_BR.po | 0 {lib/libalpm/po => po}/remove-potcdate.sin | 0 {src/pacman/po => po}/ru_RU.po | 0 scripts/Makefile.am | 44 ++-- scripts/{abs.in => abs.sh.in} | 26 +- scripts/{gensync.in => gensync.sh.in} | 44 ++-- scripts/{makepkg.in => makepkg.sh.in} | 13 +- scripts/{makeworld.in => makeworld.sh.in} | 9 +- .../{pacman-optimize.in => pacman-optimize.sh.in} | 35 +- scripts/{rankmirrors.in => rankmirrors.py.in} | 8 +- scripts/{repo-add.in => repo-add.sh.in} | 33 +- scripts/{repo-remove.in => repo-remove.sh.in} | 29 +- scripts/{updatesync.in => updatesync.sh.in} | 41 +-- src/pacman/Makefile.am | 2 - src/pacman/po/.gitignore | 6 - src/pacman/po/LINGUAS | 10 - src/pacman/po/Makefile.in.in | 355 -------------------- src/pacman/po/POTFILES.in | 12 - src/pacman/po/remove-potcdate.sin | 19 - 38 files changed, 162 insertions(+), 576 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 6b7b9743181078aa7152daffdfc1eaeb46304c0f \ a501b72e40062c9e606a787657582d1859c35d35

1 0

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-266-g6b7b974
by Dan McGee 06 Jul '07

06 Jul '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via 6b7b9743181078aa7152daffdfc1eaeb46304c0f (commit) from 59d9ccf48d84fd1e59f78cb4dcf428e53d1c6911 (commit) - Log ----------------------------------------------------------------- commit 6b7b9743181078aa7152daffdfc1eaeb46304c0f Author: Dan McGee <dan(a)archlinux.org> Date: Fri Jul 6 00:33:26 2007 -0400 Fix up pactest TODO Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: pactest/TODO | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 59d9ccf48d84fd1e59f78cb4dcf428e53d1c6911 \ 6b7b9743181078aa7152daffdfc1eaeb46304c0f

1 0

[pacman-dev] makepkg -L fails
by Andreas Radke 03 Jul '07

03 Jul '07

while makepkg without the logging works when compiling udev: http://bugs.archlinux.org/task/7507 pacman 3.0.5-1 on x86_64 AndyRTR

5 4

[pacman-dev] [PATCH] Change the exit code for pacman_deptest().
by Andrew Fyfe 02 Jul '07

02 Jul '07

Changed the exit code for missing deps from 1 to 127 because 1 is used for other errors. makepkg breaks if pacman exits with 1 for any reason other than a missing dep. Signed-off-by: Andrew Fyfe <andrew(a)neptune-one.net> --- scripts/makepkg.in | 2 +- src/pacman/deptest.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.in b/scripts/makepkg.in index 8be44c6..96c94ea 100644 --- a/scripts/makepkg.in +++ b/scripts/makepkg.in @@ -296,7 +296,7 @@ check_deps() { pmout=$(pacman $PACMAN_OPTS -T $*) ret=$? - if [ $ret -eq 1 ]; then #unresolved deps + if [ $ret -eq 127 ]; then #unresolved deps echo "$pmout" elif [ $ret -ne 0 ]; then error "$(gettext "Pacman returned a fatal error (%i): %s")" "$ret" "$pmout" diff --git a/src/pacman/deptest.c b/src/pacman/deptest.c index 264ba1c..5addfb3 100644 --- a/src/pacman/deptest.c +++ b/src/pacman/deptest.c @@ -79,7 +79,7 @@ int pacman_deptest(alpm_list_t *targets) if(!found) { printf("%s\n", saved_target); - retval = 1; + retval = 127; } free(saved_target); } -- 1.5.2.2

2 2

[pacman-dev] [GIT] pacman branch, master now at v3.0.0-265-g59d9ccf
by Dan McGee 02 Jul '07

02 Jul '07

Hello, This is an automated email from the git hooks/update script, it was generated because a ref change was pushed to the repository. Updating branch, master, via 59d9ccf48d84fd1e59f78cb4dcf428e53d1c6911 (commit) via 4d9b5d4a2a4cf96de8d6b4afa34a4d3424bf96b4 (commit) via 4d286e567b18a44715cb5dee37e7179a79cdef87 (commit) from 89ed15c9c2dd5449d50e5ec2283a4d6ebac5f328 (commit) - Log ----------------------------------------------------------------- commit 59d9ccf48d84fd1e59f78cb4dcf428e53d1c6911 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jul 2 10:06:03 2007 -0400 Reduce delay in pactest when running MODIFIED tests pactest was sleeping for 3 seconds, which is unnecessary. Reduce it to 1.5 seconds and clean up the code a bit anyway. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 4d9b5d4a2a4cf96de8d6b4afa34a4d3424bf96b4 Author: Chantry Xavier <shiningxc(a)gmail.com> Date: Mon Jul 2 11:29:05 2007 +0200 pacman/query.c : return after a query on all packages. Previously, the following error was displayed even after a valid and successful query operation on all packages, like -Q , -Ql, -Qi, -Qil : no targets specified (use -h for help) Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> commit 4d286e567b18a44715cb5dee37e7179a79cdef87 Author: Dan McGee <dan(a)archlinux.org> Date: Sun Jul 1 23:27:20 2007 -0400 Update 'submitting-patches' Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Diffstat: pactest/pmenv.py | 6 ++---- src/pacman/query.c | 1 + submitting-patches | 20 ++++++++++++++++---- 3 files changed, 19 insertions(+), 8 deletions(-) hooks/update --- Git Source Code Management System hooks/update refs/heads/master \ 89ed15c9c2dd5449d50e5ec2283a4d6ebac5f328 \ 59d9ccf48d84fd1e59f78cb4dcf428e53d1c6911

1 0