- Pacman-dev - lists.archlinux.org

Re: [pacman-dev] pacman-contrib package
by Xavier 02 Mar '08

02 Mar '08

On Sun, Mar 02, 2008 at 09:27:59PM +1000, Allan McRae wrote: > Xavier wrote: > > Arf, I was trying to make some change to that syntax file this morning. My > > changes didn't seem to have any effect, it was driving me mad. > > After a while, I figured out I had both pkgbuild.vim and PKGBUILD.vim files > > in ftdetect and syntax :P > > That's because pacman-contrib installs PKGBUILD.vim, but Nathan's > > instructions which I followed previously used pkgbuild.vim : > > http://www.archlinux.org/pipermail/pacman-dev/2008-January/010782.html > > > > Anyway, both license=('foo') and license=foo work fine now. > > > > > Great. I tried but seemed to make things worse! Hmm, I just realized what I said wasn't clear. The consequence of what I said above is that I had both PKGBUILD.vim and pkgbuild.vim. I was editing the PKGBUILD.vim one, but it was actually pkgbuild.vim which was used. So my changes were ignored. > > But I think only the first should be supported (the man page says that > > license is an array), so I updated PKGBUILD.vim accordingly > I agree. license=foo is technically wrong so we should not support it. > I think of it as as subtle way to improve PKGBUILD quality. > Ok good.

2 1

Re: [pacman-dev] pacman-contrib package
by Xavier 02 Mar '08

02 Mar '08

On Sat, Feb 23, 2008 at 10:36:35PM +1000, Allan McRae wrote: > Xavier wrote: > > About PKGBUILD syntax, it doesn't seem to understand fields like this : > > license=('foo') > > only license=foo > > > > In the second case, it will highlight foo because it doesn't exist. But not > > in the first case. > > Same for arch fields, and maybe others. > Hmm, I will actually install vim and take a look at the syntax file and > see if I can improve it. Of course, if someone who know vim syntax > files and wants to fix it.... > > Arf, I was trying to make some change to that syntax file this morning. My changes didn't seem to have any effect, it was driving me mad. After a while, I figured out I had both pkgbuild.vim and PKGBUILD.vim files in ftdetect and syntax :P That's because pacman-contrib installs PKGBUILD.vim, but Nathan's instructions which I followed previously used pkgbuild.vim : http://www.archlinux.org/pipermail/pacman-dev/2008-January/010782.html Anyway, both license=('foo') and license=foo work fine now. But I think only the first should be supported (the man page says that license is an array), so I updated PKGBUILD.vim accordingly.

2 1

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.1.2-16-gaecc2fd
by Dan McGee 01 Mar '08

01 Mar '08

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 aecc2fd19037d0580df9600628acace93cb0dd46 (commit) via 7613f2e21aa4df72e7a520b2cd80c592a0cb63a2 (commit) via d49f42ba752b3ecb3cf02b0b137fc422bfd7c76c (commit) from 8efe0ecb250050e9061854476468298afb5944d2 (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: contrib/PKGBUILD.vim | 6 ++++++ lib/libalpm/package.c | 2 +- scripts/makepkg.sh.in | 2 +- 3 files changed, 8 insertions(+), 2 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] testpkg rework
by Nagy Gabor 01 Mar '08

01 Mar '08

Hi! Patch attached. Bye ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

2 2

[pacman-dev] [PATCH] Set a missing pm_errno in _alpm_pkg_load()
by Nagy Gabor 01 Mar '08

01 Mar '08

Hi! I found a little issue during testpkg testing. Patch attached. Bye ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

1 0

[pacman-dev] multiple replacers
by Nagy Gabor 29 Feb '08

29 Feb '08

Hi! I would like to warm up this thread again: http://www.archlinux.org/pipermail/pacman-dev/2008-January/010877.html What do you think? Bye ---------------------------------------------------- SZTE Egyetemi Könyvtár - http://www.bibl.u-szeged.hu This mail sent through IMP: http://horde.org/imp/

2 1

[pacman-dev] Pharmacy Online February 70% OFF
by pacman-dev＠archlinux.org 27 Feb '08

27 Feb '08

4 3

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.1.2-67-gd75f693
by Dan McGee 26 Feb '08

26 Feb '08

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 d75f693155c3d7e595a779f15beea516d321f228 (commit) via 8efe0ecb250050e9061854476468298afb5944d2 (commit) via 7a9d444de874f7e814b36711d3cfc51224f3385c (commit) via 66591e8284054adcefb416a5331916a6812582a8 (commit) via 7eaad2f2a9879499d29e20bb897b018f0b13f6c1 (commit) via c23ecc6160749e635767cdfbd23760e9e809c949 (commit) via f159203f6fb217bb7bed5ffea8c2518325a9ec12 (commit) via 81a2a06818d367f8528c74311171417beb9e1592 (commit) via 3e8ae774bdbc8572613a22988476b6b4e7ef91fd (commit) via d5857ee15b28af5de57614348f930046e122ba85 (commit) via 271ecb8bfc487383256b8247d6e6d8217a8a2ff0 (commit) from 143135e666f5564240868810f0b8f4bc46ff87b5 (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 d75f693155c3d7e595a779f15beea516d321f228 Merge: f159203f6fb217bb7bed5ffea8c2518325a9ec12 8efe0ecb250050e9061854476468298afb5944d2 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Feb 25 20:49:24 2008 -0600 Merge branch 'maint' Also bump the devel version on the master branch to 3.2.0devel. commit f159203f6fb217bb7bed5ffea8c2518325a9ec12 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jan 22 22:49:45 2008 -0600 Remove pmserver_t abstraction Remove what was a pretty weird abstraction in the libalpm backend. Instead of parsing server URLs as we get them (of which we don't usually use more than a handful anyway), wait until they are actually used, which allows us to store them as a simple string list instead. This allows us to remove a lot of code, and will greatly simplify the continuing refactoring of the download code. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 81a2a06818d367f8528c74311171417beb9e1592 Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jan 22 22:00:12 2008 -0600 Add new stub download functions for use throughout the code Add new stub functions that work by calling the existing (terrible) download forreal function, which needs a serious overhaul. Hide the existing functions and switch all former users to the new functions. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 3e8ae774bdbc8572613a22988476b6b4e7ef91fd Author: Dan McGee <dan(a)archlinux.org> Date: Tue Jan 22 21:36:11 2008 -0600 Move download code out of server.c This is the first in what will be a series of patches to clean up the current download code in libalpm. Start by moving download code out of server.c and into download.c. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: TRANSLATORS | 35 +++---- configure.ac | 6 +- lib/libalpm/Makefile.am | 2 +- lib/libalpm/alpm.h | 1 - lib/libalpm/db.c | 37 ++----- lib/libalpm/{server.c => dload.c} | 188 ++++++++++++----------------------- lib/libalpm/{add.h => dload.h} | 27 +++-- lib/libalpm/handle.c | 1 - lib/libalpm/po/Makevars | 4 +- lib/libalpm/po/POTFILES.in | 2 +- lib/libalpm/po/pl.po | 4 +- lib/libalpm/po/zh_CN.po | 18 ++-- lib/libalpm/server.h | 46 --------- lib/libalpm/sync.c | 8 +- po/Makevars | 4 +- po/es.po | 4 +- po/zh_CN.po | 201 +++++++++++++++++++----------------- src/pacman/add.c | 4 +- src/pacman/remove.c | 4 +- 19 files changed, 241 insertions(+), 355 deletions(-) rename lib/libalpm/{server.c => dload.c} (71%) copy lib/libalpm/{add.h => dload.h} (60%) delete mode 100644 lib/libalpm/server.h hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v3.1.2-13-g8efe0ec
by Dan McGee 26 Feb '08

26 Feb '08

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 8efe0ecb250050e9061854476468298afb5944d2 (commit) via 7a9d444de874f7e814b36711d3cfc51224f3385c (commit) via 66591e8284054adcefb416a5331916a6812582a8 (commit) via 7eaad2f2a9879499d29e20bb897b018f0b13f6c1 (commit) via c23ecc6160749e635767cdfbd23760e9e809c949 (commit) from d5857ee15b28af5de57614348f930046e122ba85 (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: configure.ac | 4 +- lib/libalpm/po/Makevars | 4 +- lib/libalpm/po/pl.po | 4 +- lib/libalpm/po/zh_CN.po | 18 ++-- po/Makevars | 4 +- po/es.po | 4 +- po/zh_CN.po | 201 +++++++++++++++++++++++++---------------------- src/pacman/add.c | 4 +- src/pacman/remove.c | 4 +- 9 files changed, 127 insertions(+), 120 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] xgettext : change pass-c-format flag to c-format.
by Chantry Xavier 25 Feb '08

25 Feb '08

Currently xgettext apparently attempts to autodetect c format strings (eg a string with a %s) to decide whether to use c-format flag or not. If we use --flag=_:1:c-format instead of --flag=_:1:pass-c-format, the c-format will be applied everywhere. I couldn't find this documented anywhere though. But the pass prefix is mentioned here : http://www.gnu.org/software/gettext/manual/html_node/xgettext-Invocation.ht… "Specifies additional flags for strings occurring as part of the argth argument of the function word. The possible flags are the possible format string indicators, such as ‘c-format’, and their negations, such as ‘no-c-format’, possibly prefixed with ‘pass-’." And c-format is documented there : http://www.gnu.org/software/gettext/manual/html_node/c_002dformat-Flag.html… "This situation happens quite often. The printf function is often called with strings which do not contain a format specifier. Of course one would normally use fputs but it does happen. In this case xgettext does not recognize this as a format string but what happens if the translation introduces a valid format specifier? The printf function will try to access one of the parameters but none exists because the original code does not pass any parameters." And that's exactly what happened with FS#9658. So using c-format for every string will prevent this issue from happening again. Signed-off-by: Chantry Xavier <shiningxc(a)gmail.com> --- lib/libalpm/po/Makevars | 4 ++-- po/Makevars | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/libalpm/po/Makevars b/lib/libalpm/po/Makevars index 1b1b827..d751522 100644 --- a/lib/libalpm/po/Makevars +++ b/lib/libalpm/po/Makevars @@ -9,8 +9,8 @@ top_builddir = ../../../ # These options get passed to xgettext. XGETTEXT_OPTIONS = \ - --keyword=_ --flag=_:1:pass-c-format \ - --keyword=N_ --flag=N_:1:pass-c-format + --keyword=_ --flag=_:1:c-format \ + --keyword=N_ --flag=N_:1:c-format # This is the copyright holder that gets inserted into the header of the # $(DOMAIN).pot file. Set this to the copyright holder of the surrounding diff --git a/po/Makevars b/po/Makevars index 94d8294..38a4692 100644 --- a/po/Makevars +++ b/po/Makevars @@ -9,8 +9,8 @@ top_builddir = ../ # These options get passed to xgettext. XGETTEXT_OPTIONS = \ - --keyword=_ --flag=_:1:pass-c-format \ - --keyword=N_ --flag=N_:1:pass-c-format + --keyword=_ --flag=_:1:c-format \ + --keyword=N_ --flag=N_:1:c-format # This is the copyright holder that gets inserted into the header of the # $(DOMAIN).pot file. -- 1.5.4.2

1 0