Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-08 Thread Nagy Gabor
Here are some output of the patch just sent to the ML. $ pacman -Sp --print-format %r/%n-%v : %l [%s] kdelibs extra/qt-4.5.2-6 : http://mir1.archlinuxfr.org/archlinux/extra/os/i686/qt-4.5.2-6-i686.pkg.tar.gz [28914122] extra/ilmbase-1.0.1-1 :

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-08 Thread Xavier
On Tue, Sep 8, 2009 at 1:19 PM, Nagy Gaborng...@bibl.u-szeged.hu wrote: Here are some output of the patch just sent to the ML. $ pacman -Sp --print-format %r/%n-%v : %l [%s] kdelibs extra/qt-4.5.2-6 : http://mir1.archlinuxfr.org/archlinux/extra/os/i686/qt-4.5.2-6-i686.pkg.tar.gz [28914122]

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-08 Thread Xyne
Here are some output of the patch just sent to the ML. $ pacman -Sp --print-format %r/%n-%v : %l [%s] kdelibs extra/qt-4.5.2-6 : http://mir1.archlinuxfr.org/archlinux/extra/os/i686/qt-4.5.2-6-i686.pkg.tar.gz [28914122] extra/ilmbase-1.0.1-1 :

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-07 Thread Xavier
On Fri, Sep 4, 2009 at 3:50 AM, Xynex...@archlinux.ca wrote: Instead of adding several options, wouldn't it be possible to let -p accept a string argument with formatting information similar to the way the date command works. This would be extensible in the future and could include the

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-05 Thread Xyne
Instead of adding several options, wouldn't it be possible to let -p accept a string argument with formatting information similar to the way the date command works. This would be extensible in the future and could include the following interpretted sequences (among others) %u - url

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-05 Thread Xavier
On Sat, Sep 5, 2009 at 10:16 PM, Xynex...@archlinux.ca wrote: I think using local for %r is a good idea. As for %u, maybe it should be changed to % l for location. It could return the full path to the package archive when using -U. It's relatively useless for -R but I agree that it should

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-04 Thread Xavier
On Fri, Sep 4, 2009 at 3:50 AM, Xynex...@archlinux.ca wrote: Instead of adding several options, wouldn't it be possible to let -p accept a string argument with formatting information similar to the way the date command works. This would be extensible in the future and could include the

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-04 Thread Xavier
On Fri, Sep 4, 2009 at 3:49 AM, Xynex...@archlinux.ca wrote: On Thu, 3 Sep 2009 15:20:03 +0200 Xavier shinin...@gmail.com wrote: I have a preliminary patch for this. I am not really happy with the C code I wrote for doing the substitutions, but it works. Does it make any sense to have this

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-03 Thread Xyne
Instead of adding several options, wouldn't it be possible to let -p accept a string argument with formatting information similar to the way the date command works. This would be extensible in the future and could include the following interpretted sequences (among others) %u - url %n -

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-02 Thread Xavier
On Wed, Sep 2, 2009 at 5:16 AM, Xynex...@archlinux.ca wrote: --print-pkg and --print-uris looks good to me but would you keep your new feature on top of these two options : --print-pkg and --print-uris only show non-download packages --print-pkg --print-pkg and --print-uris --print-uris show

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-02 Thread Laszlo Papp
Instead of adding several options, wouldn't it be possible to let -p accept a string argument with formatting information similar to the way the date command works. This would be extensible in the future and could include the following interpretted sequences (among others) %u - url

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-09-01 Thread Xyne
--print-pkg and --print-uris looks good to me but would you keep your new feature on top of these two options : --print-pkg and --print-uris only show non-download packages --print-pkg --print-pkg and --print-uris --print-uris show all ? an alternative : what if we added the download

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Xavier
On Mon, Aug 31, 2009 at 5:45 AM, Laszlo Pappdjsza...@gmail.com wrote: Hello Xavier! I didn't have so much time at the weekend, but as I saw maybe it would be better to choose a more talkative name for it, e.g. --print-package or something (it's just an example). What will happen if there

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Laszlo Papp
Best Regards, Laszlo Papp On Mon, Aug 31, 2009 at 10:59 AM, Xavier shinin...@gmail.com wrote: On Mon, Aug 31, 2009 at 5:45 AM, Laszlo Pappdjsza...@gmail.com wrote: Hello Xavier! I didn't have so much time at the weekend, but as I saw maybe it would be better to choose a more talkative

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Xavier
On Mon, Aug 31, 2009 at 8:47 PM, Laszlo Pappdjsza...@gmail.com wrote: --print-pkg and --print-uris looks good to me Me too, but it would be nice to keep the older -Sp option for --print-uris, because everybody know it so :) I didn't mean to drop it but would you keep your new feature on

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Laszlo Papp
On Mon, Aug 31, 2009 at 9:04 PM, Xavier shinin...@gmail.com wrote: On Mon, Aug 31, 2009 at 8:47 PM, Laszlo Pappdjsza...@gmail.com wrote: --print-pkg and --print-uris looks good to me Me too, but it would be nice to keep the older -Sp option for --print-uris, because everybody know

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Xavier
On Mon, Aug 31, 2009 at 9:15 PM, Laszlo Pappdjsza...@gmail.com wrote: But since --print-pkgs is only a long option, we would need to use --print-pkg --print-pkg which is very ugly ... Is it clearer now ? --print-pkgs seems fine for me, the later one isn't so beautiful in fact, but when i

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-31 Thread Laszlo Papp
On Mon, Aug 31, 2009 at 9:19 PM, Xavier shinin...@gmail.com wrote: On Mon, Aug 31, 2009 at 9:15 PM, Laszlo Pappdjsza...@gmail.com wrote: But since --print-pkgs is only a long option, we would need to use --print-pkg --print-pkg which is very ugly ... Is it clearer now ? --print-pkgs

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-30 Thread Laszlo Papp
On Fri, Aug 28, 2009 at 12:52 AM, Xavier shinin...@gmail.com wrote: On Fri, Aug 28, 2009 at 12:31 AM, Laszlo Pappdjsza...@gmail.com wrote: Signed-off-by: Laszlo Papp djsza...@gmail.com --- src/pacman/pacman.c |2 +- src/pacman/sync.c |3 +++ 2 files changed, 4 insertions(+),

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-28 Thread Nagy Gabor
+ if ((config-op_s_printuris = 2) !alpm_pkg_download_size(pkg)) { + continue; +} This means that -Sp prints everything, -Spp prints only the to-be-download URIs. This is good for backward compatibility, but for my taste -Sp should print only the to-be-download URIs, and -Spp all.

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-28 Thread Laszlo Papp
On Fri, Aug 28, 2009 at 11:34 AM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: + if ((config-op_s_printuris = 2) !alpm_pkg_download_size(pkg)) { + continue; +} This means that -Sp prints everything, -Spp prints only the to-be-download URIs. This is good for backward compatibility, but

[pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-27 Thread Laszlo Papp
Signed-off-by: Laszlo Papp djsza...@gmail.com --- src/pacman/pacman.c |2 +- src/pacman/sync.c |3 +++ 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c index 25647b5..00b14f5 100644 --- a/src/pacman/pacman.c +++

Re: [pacman-dev] [PATCH] '-Spp and -Supp' options were added to generate uri list without the downloaded package uris

2009-08-27 Thread Xavier
On Fri, Aug 28, 2009 at 12:31 AM, Laszlo Pappdjsza...@gmail.com wrote: Signed-off-by: Laszlo Papp djsza...@gmail.com ---  src/pacman/pacman.c |    2 +-  src/pacman/sync.c   |    3 +++  2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/pacman/pacman.c b/src/pacman/pacman.c