- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] Reduce unnecessary get_name() function calls
by Dan McGee 21 Oct '09

21 Oct '09

alpm_pkg_get_name() gives us little benefit in backend code besides a NULL check on the package passed in; we could do that ourself if necessary. By changing to direct references in the cases where we are sure we have a valid package, we save a function call each time we need a package name. This function can't be inlined because it is externally accessible. This cuts the calls to get_name() from 1.3 million times in a pacman -Qu operation to around 2400. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- lib/libalpm/package.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/package.c b/lib/libalpm/package.c index 572b863..83a2fb8 100644 --- a/lib/libalpm/package.c +++ b/lib/libalpm/package.c @@ -573,7 +573,7 @@ alpm_list_t SYMEXPORT *alpm_pkg_compute_requiredby(pmpkg_t *pkg) } pmpkg_t *cachepkg = i->data; if(_alpm_dep_edge(cachepkg, pkg)) { - const char *cachepkgname = alpm_pkg_get_name(cachepkg); + const char *cachepkgname = pkg->name; reqs = alpm_list_add(reqs, strdup(cachepkgname)); } } @@ -896,7 +896,7 @@ int _alpm_pkg_cmp(const void *p1, const void *p2) { pmpkg_t *pkg1 = (pmpkg_t *)p1; pmpkg_t *pkg2 = (pmpkg_t *)p2; - return(strcmp(alpm_pkg_get_name(pkg1), alpm_pkg_get_name(pkg2))); + return(strcmp(pkg1->name, pkg2->name)); } /* Test for existence of a package in a alpm_list_t* @@ -915,7 +915,7 @@ pmpkg_t *_alpm_pkg_find(alpm_list_t *haystack, const char *needle) for(lp = haystack; lp; lp = lp->next) { pmpkg_t *info = lp->data; - if(info && strcmp(alpm_pkg_get_name(info), needle) == 0) { + if(info && strcmp(info->name, needle) == 0) { return(info); } } -- 1.6.5.1

1 0

[pacman-dev] [PATCH] makepkg: fix testing for built package presence
by Allan McRae 21 Oct '09

21 Oct '09

Commit c7e4d10d introduced a small error in the testing of whether a package is already built. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- scripts/makepkg.sh.in | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 989b252..7e10eef 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1724,7 +1724,7 @@ pkgbase=${pkgbase:-${pkgname[0]}} if [ "$SPLITPKG" -eq 0 ]; then if [ $ -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ - -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-any${PKGEXT}" $ \ + -o -f "$PKGDEST/${pkgname}-${pkgver}-${pkgrel}-any${PKGEXT}" \) \ -a "$FORCE" -eq 0 -a "$SOURCEONLY" -eq 0 -a "$NOBUILD" -eq 0 ]; then if [ "$INSTALL" -eq 1 ]; then warning "$(gettext "A package has already been built, installing existing package...")" @@ -1740,7 +1740,7 @@ else somepkgbuilt=0 for pkg in ${pkgname[@]}; do if [ $ -f "$PKGDEST/${pkg}-${pkgver}-${pkgrel}-${CARCH}${PKGEXT}" \ - -f "$PKGDEST/${pkg}-${pkgver}-${pkgrel}-any${PKGEXT}" $ ]; then + -o -f "$PKGDEST/${pkg}-${pkgver}-${pkgrel}-any${PKGEXT}" \) ]; then somepkgbuilt=1 else allpkgbuilt=0 -- 1.6.5.1

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.3.2-5-g5ffc1ad
by dan＠archlinux.org 21 Oct '09

21 Oct '09

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, maint has been updated via 5ffc1ad3cdb1ec252739c62c9d614aba468bb22c (commit) from da5fb3ee4f66dbdf0620bbb087fd759ddc9ae6e0 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- ----------------------------------------------------------------------- Summary of changes: po/zh_CN.po | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.3.2-76-g6c635d7
by dan＠archlinux.org 21 Oct '09

21 Oct '09

This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The official pacman repository". The branch, master has been updated via 6c635d76a042add78f65f732928c2f10c2e5567a (commit) via c7e4d10df378ec12e9e47bb05120778ea98015e1 (commit) via 2020e6297b021946c31c1e46b6ede3c4734ef85d (commit) via 2c2596177d88a60cbef8154927f847ca214aadd0 (commit) via 3d67d9b16c53fe70c1b7149b5a6bfbbf17522ba3 (commit) via 5ffc1ad3cdb1ec252739c62c9d614aba468bb22c (commit) via 4281a1a7f2d24cfcf0997122cc75c216a17f3e5b (commit) via da5fb3ee4f66dbdf0620bbb087fd759ddc9ae6e0 (commit) from f9582c7df223e1eb4078e6db4e2d35dc0f2bf9be (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 6c635d76a042add78f65f732928c2f10c2e5567a Merge: c7e4d10df378ec12e9e47bb05120778ea98015e1 5ffc1ad3cdb1ec252739c62c9d614aba468bb22c Author: Dan McGee <dan(a)archlinux.org> Date: Tue Oct 20 22:29:20 2009 -0500 Merge branch 'maint' commit c7e4d10df378ec12e9e47bb05120778ea98015e1 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Oct 12 00:13:16 2009 +1000 makepkg: allow overriding arch in split packages This allows building a mixture of binary and arch=any packages. Fixes FS#15955. The value of CARCH is no longer overridden to "any" in when arch=any is used and the assigning of the "any" arch is delayed to during the packaging stage. Adjustments were required to fix installing and checking for pre-built packages of varing arches. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 2020e6297b021946c31c1e46b6ede3c4734ef85d Author: Allan McRae <allan(a)archlinux.org> Date: Sun Oct 11 22:32:22 2009 +1000 makepkg: allow overriding pkgver and pkgrel in split packages Fixing a single package within a split package requires the overriding or pkgrel. In very rare (but existing) cases, it is useful to override pkgver. Partial fix for FS#15955. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 2c2596177d88a60cbef8154927f847ca214aadd0 Author: Xavier Chantry <shiningxc(a)gmail.com> Date: Mon Oct 12 23:05:36 2009 +0200 dload.c : clear sigaction flag to make valgrind happy This fixes the following valgrind warning : ==26831== Syscall param rt_sigaction(act->sa_flags) points to uninitialised byte(s) ==26831== at 0x4282547: __libc_sigaction (in /lib/libc-2.10.1.so) ==26831== by 0x403C693: download_internal (dload.c:152) ==26831== by 0x403D0E4: _alpm_download_single_file (dload.c:311) ==26831== by 0x4033B72: alpm_db_update (be_files.c:319) ==26831== by 0x805205E: pacman_sync (sync.c:257) ==26831== by 0x804EE54: main (pacman.c:1120) ==26831== Address 0xbec6cc04 is on thread 1's stack ==26831== ==26831== Syscall param rt_sigaction(act->sa_restorer) points to uninitialised byte(s) ==26831== at 0x4282547: __libc_sigaction (in /lib/libc-2.10.1.so) ==26831== by 0x403C693: download_internal (dload.c:152) ==26831== by 0x403D0E4: _alpm_download_single_file (dload.c:311) ==26831== by 0x4033B72: alpm_db_update (be_files.c:319) ==26831== by 0x805205E: pacman_sync (sync.c:257) ==26831== by 0x804EE54: main (pacman.c:1120) ==26831== Address 0xbec6cc08 is on thread 1's stack ==26831== Signed-off-by: Xavier Chantry <shiningxc(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3d67d9b16c53fe70c1b7149b5a6bfbbf17522ba3 Author: Cedric Staniewski <cedric(a)gmx.ca> Date: Tue Oct 20 15:44:00 2009 +0200 makepkg, repo-add: replace external commands with bash substitutions where possible This also removes the awk dependency from makepkg and repo-add. Signed-off-by: Cedric Staniewski <cedric(a)gmx.ca> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 4281a1a7f2d24cfcf0997122cc75c216a17f3e5b Author: Laszlo Papp <djszapi(a)archlinux.us> Date: Sat Oct 17 23:12:19 2009 +0200 Size handling was changed in fgets() functions Pacman's fgets function in the API used hardcoded numbers to identify the size. This is not good practice, so replace them with sizeof handling. Signed-off-by: Laszlo Papp <djszapi(a)archlinux.us> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: PKGBUILD-split.proto | 3 ++ doc/PKGBUILD.5.txt | 6 ++-- lib/libalpm/be_files.c | 49 +++++++++++++------------- lib/libalpm/dload.c | 1 + lib/libalpm/trans.c | 3 +- po/el.po | 8 ++-- po/zh_CN.po | 4 +- scripts/makepkg.sh.in | 91 +++++++++++++++++++++++++++-------------------- scripts/repo-add.sh.in | 6 ++- src/pacman/util.c | 3 +- 10 files changed, 98 insertions(+), 76 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] zh_CN issues in makepkg
by Allan McRae 20 Oct '09

20 Oct '09

Hi, Currently makepkg can not run in zh_CN due to (at least) this translated string: msgid "Extracting %s with %s" msgstr "正在使用 %2$s 解压缩 %1$s" The issue is due to bash not being able to handle positional parameters (1$ and 2$) as "$" are special characters to bash. Is there a way to translate this string without changing the order of the strings? Allan

2 2

[pacman-dev] [PATCH] makepkg: use basename instead of a sed command
by Cedric Staniewski 20 Oct '09

20 Oct '09

The basename command is easier to understand than a sed command and it is even slightly faster than sed. Signed-off-by: Cedric Staniewski <cedric(a)gmx.ca> --- or if you prefer bashisms, see next patch scripts/makepkg.sh.in | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 24fddf6..62d0923 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -190,7 +190,7 @@ get_filename() { # if a filename is specified, use it local filename=$(echo $1 | sed 's|::.*||') # if it is just an URL, we only keep the last component - echo "$filename" | sed 's|^.*://.*/||g' + basename "$filename" } # extract the URL from a source entry @@ -321,7 +321,7 @@ download_file() { # destination file local file=$3 # temporary download file, default to last component of the URL - local dlfile=$(echo "$url" | sed 's|^.*://.*/||g') + local dlfile=$(basename "$url") # replace %o by the temporary dlfile if it exists if echo "$dlcmd" | grep -q "%o" ; then -- 1.6.5

4 12

[pacman-dev] Is -Suu supposed to also update packages?
by Cedric Staniewski 20 Oct '09

20 Oct '09

Hi, while implementing pacman's downgrade operation in bash, I noticed that pacman also includes upgradeable package in the list of downgradeable ones. I did not complete the downgrade, but I would assume that it would have actually updated some of the packages. In my bash function I filter the upgradeable packages out of the installed ones and this is what I expected pacman would do, too. Apparently, this is not the case and I wonder, if this is the intended behavior or if it should be considered as a bug. According to the documentation, I would say this is intended, so the term "downgrade" is a little bit misleading in my opinion. > Pass this option twice to enable package downgrade; in this case pacman will select sync packages whose version does not match with > the local version. This can be useful when the user switches from a testing repo to a stable one. Thanks for clarification, Cedric

2 2

[pacman-dev] [PATCH] makepkg: fix --allsource when remote source files do not reside in SRCDEST
by Cedric Staniewski 20 Oct '09

20 Oct '09

When using the --allsource option, it was assumed that remote source files reside in SRCDEST which is not necessarily always the case. Signed-off-by: Cedric Staniewski <cedric(a)gmx.ca> --- scripts/makepkg.sh.in | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 24fddf6..806a325 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1074,9 +1074,10 @@ create_srcpackage() { if [ -f "$netfile" ]; then msg2 "$(gettext "Adding %s...")" "$netfile" ln -s "${startdir}/$netfile" "${srclinks}/${pkgbase}" - elif [ "$SOURCEONLY" -eq 2 -a -f "$SRCDEST/$file" ]; then + elif [ "$SOURCEONLY" -eq 2 ] && [ -f "$file" -o -f "$SRCDEST/$file" ]; then msg2 "$(gettext "Adding %s...")" "$file" - ln -s "$SRCDEST/$file" "${srclinks}/${pkgbase}/" + [ ! -f "$file" ] && file="$SRCDEST/$file" + ln -s "$file" "${srclinks}/${pkgbase}/" fi done -- 1.6.5

4 5

[pacman-dev] Build issues with the maint branch of pacman
by Ciprian Dorin, Craciun 20 Oct '09

20 Oct '09

Hello all! While trying to build (a patched version) of the latest maint branch in the Git repository I get the following error: ~~~~ $ autoconf configure.ac:60: error: possibly undefined macro: AM_INIT_AUTOMAKE If this token and others are legitimate, please use m4_pattern_allow. See the Autoconf documentation. configure.ac:128: error: possibly undefined macro: AC_PROG_LIBTOOL configure.ac:132: error: possibly undefined macro: AM_GNU_GETTEXT configure.ac:133: error: possibly undefined macro: AM_GNU_GETTEXT_VERSION configure.ac:155: error: possibly undefined macro: AM_CONDITIONAL ~~~~ (My patch is only adding a git-dlagent script, and is not touching otherwise the configure.ac file.) Any ideas? (I'm using the latest up-to-date version of ArchLinux.) Thanks, Ciprian.

3 13

[pacman-dev] makepkg test suite
by Allan McRae 20 Oct '09

20 Oct '09

Hi all, I am starting to write a test suite for makepkg. Hopefully this will make it more difficult to break things in the future... I have written a brief outline of how I have started implementing this here: http://wiki.archlinux.org/index.php/User:Allan/Makepkg_Test_Suite I realize that most issues with the implementation will be discovered as it goes on but any comments at this stage would be useful. Allan

2 3