Re: [pacman-dev] little translation error

2012-02-27 Thread Xavier Chantry
2012/2/26 solsTiCe d'Hiver solstice.dhi...@gmail.com: hi. I have sent a message about that to shining on transifex, but it seems to go unnoticed so I am reposting it here Sorry, I did not realize it was a private message. The advantage of transifex is that everyone can contribute directly to

Re: [pacman-dev] [PATCH] Handle removal of empty directories properly

2011-07-17 Thread Xavier Chantry
On Sat, Jul 16, 2011 at 5:11 PM, Dan McGee d...@archlinux.org wrote: This addresses FS#25141. We shouldn't remove every empty directory we come across during the removal process unless it is truly not known to any other package. This will prevent removal of essential directories such as

Re: [pacman-dev] [PATCH] Simplify alpm_list_previous

2011-07-04 Thread Xavier Chantry
On Mon, Jul 4, 2011 at 4:16 AM, Allan McRae al...@archlinux.org wrote: We can readily detect the first node in a list by checking if node-prev-next is NULL. So there is no need to pass the head of the list to this function and its prototype now looks like all the other item accessors. This

Re: [pacman-dev] [PATCH v2] makepkg: Add support for verifying pgp signatures

2011-07-04 Thread Xavier Chantry
On Mon, Jul 4, 2011 at 4:36 PM, Allan McRae al...@archlinux.org wrote: I still wonder if --skippgpcheck is too long, but I can not think of a better name.  Suggestions from anyone? --skipinteg / --skipsig or --skippgp ? +*\--skippgpcheck*:: +       Verify PGP signatures of the source

Re: [pacman-dev] [PATCH] Remove old TODO lists

2011-07-03 Thread Xavier Chantry
On Wed, Jun 22, 2011 at 10:54 AM, Allan McRae al...@archlinux.org wrote: These had not been touched since 2007 and had lost most of their relevance.  The bug tracker is a better place for the filing of ideas. Agreed, but if some items are not completely obsolete and don't appear anywhere in

Re: [pacman-dev] [PATCH] Remove incorrect output when downloading only

2011-06-06 Thread Xavier
Allan McRae wrote: diff --git a/lib/libalpm/deps.c b/lib/libalpm/deps.c index c5fb92e..b03cd48 100644 --- a/lib/libalpm/deps.c +++ b/lib/libalpm/deps.c @@ -570,8 +570,12 @@ static pmpkg_t *resolvedep(pmdepend_t *dep, alpm_list_t *dbs, if(_alpm_pkg_should_ignore(pkg))

Re: [pacman-dev] Thoughts on database initialization and loading process

2011-04-17 Thread Xavier Chantry
On Sun, Apr 17, 2011 at 9:32 AM, Rémy Oudompheng remyoudomph...@gmail.com wrote: We are looking at a broader problem: it is actually impossible to do any sane return value checking for all functions returning an alpm_list_t* since NULL is used both as the error value and as the empty list.

Re: [pacman-dev] alpm_add_pkg() stealing ownership of allocated objects

2011-04-12 Thread Xavier
Rémy Oudompheng wrote: Hello, While doing experiments, I discovered that alpm_add_pkg() seemingly stole the ownership of a pmpkg_t previously allocated by alpm_pkg_load(). I am a bit confused here. I think the documentation can definitely be improved here: can someone confirm the following

Re: [pacman-dev] [PATCH] Remove dead stores reported by clang-analyzer

2011-04-04 Thread Xavier
Allan McRae wrote: Please send patches to the list so that we can complain about issues like: aba191d5 (Xavier Chantry2011-04-03 10:28:09 +0200 276) printf(%lc, *p); util.c: In function ‘indentprint’: util.c:276:3: error: format ‘%lc’ expects argument of type ‘wint_t

Re: [pacman-dev] [PATCH] Fix -Wshadow warnings

2011-04-04 Thread Xavier
Allan McRae wrote: But do we really want to add all these warnings in there? I use all of these: -Wclobbered -Wempty-body -Wfloat-equal -Wignored-qualifiers -Wmissing-declarations -Wmissing-parameter-type -Wmissing-prototypes -Wold-style-declaration -Woverride-init -Wsign-compare

Re: [pacman-dev] [PATCH] Remove dead stores reported by clang-analyzer

2011-04-03 Thread Xavier Chantry
as they looked trivial enough, but maybe I should have. http://code.toofishes.net/cgit/xavier/pacman.git/log/?h=working

Re: [pacman-dev] gnu89-inline

2011-04-03 Thread Xavier Chantry
On Sun, Apr 3, 2011 at 6:55 AM, Sebastian Nowicki seb...@gmail.com wrote: Looks like there are more warnings when using gnu89: $ make 21 | grep 'warning:' | cut -d' ' -f3- | sort | uniq -c     74 comma at end of enumerator list      1 initializer element is not computable at load time      7

Re: [pacman-dev] gnu89-inline

2011-04-03 Thread Xavier Chantry
On Sun, Apr 3, 2011 at 10:04 AM, Xavier Chantry chantry.xav...@gmail.com wrote: On Sun, Apr 3, 2011 at 6:55 AM, Sebastian Nowicki seb...@gmail.com wrote: Looks like there are more warnings when using gnu89: $ make 21 | grep 'warning:' | cut -d' ' -f3- | sort | uniq -c     74 comma at end

[pacman-dev] [PATCH 1/2] configure.ac: we use fabs now so -lm is needed

2011-04-02 Thread Xavier Chantry
Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- configure.ac |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/configure.ac b/configure.ac index 3299302..9fb9cb0 100644 --- a/configure.ac +++ b/configure.ac @@ -131,6 +131,9 @@ AC_PATH_PROGS([BASH_SHELL], [bash

[pacman-dev] [PATCH 2/2] dload: dont forget to initialize open_mode !

2011-04-02 Thread Xavier Chantry
mode. Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- lib/libalpm/dload.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index d9e9488..4ebc868 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -159,7 +159,8

[pacman-dev] gnu89-inline

2011-04-02 Thread Xavier Chantry
I am just curious, what do we need this flag for ? I only found this : http://gcc.gnu.org/onlinedocs/gcc/Inline.html GCC implements three different semantics of declaring a function inline. One is available with -std=gnu89 or -fgnu89-inline or when gnu_inline attribute is present on all inline

Re: [pacman-dev] gnu89-inline

2011-04-02 Thread Xavier Chantry
On Sat, Apr 2, 2011 at 7:49 PM, Dan McGee dpmc...@gmail.com wrote: On Sat, Apr 2, 2011 at 12:17 PM, Xavier Chantry chantry.xav...@gmail.com wrote: I am just curious, what do we need this flag for ? Searching commits, it wasn't too hard to find this, although I don't know the full context

[pacman-dev] [PATCH 3/7] libalpm: switch default VerifySig to Optional

2011-04-02 Thread Xavier Chantry
Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- lib/libalpm/alpm.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index b08191d..81877cc 100644 --- a/lib/libalpm/alpm.h +++ b/lib/libalpm/alpm.h @@ -254,8 +254,8 @@ int

[pacman-dev] [PATCH 7/7] pacman/query: correctly handle root files with -Qo

2011-04-02 Thread Xavier Chantry
spotted by clang-analyzer (strcmp with NULL rpath is bad) Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- src/pacman/query.c | 50 +- 1 files changed, 33 insertions(+), 17 deletions(-) diff --git a/src/pacman/query.c b/src/pacman

Re: [pacman-dev] libalpm linked against libarchive/libfetch deps (was: SyncFirst and dependencies)

2011-03-20 Thread Xavier Chantry
On Sat, Mar 19, 2011 at 9:27 PM, Lukas Fleischer archli...@cryptocrack.de wrote: Yes, this definitely is a libtool issue. It just seems to link all indirect library dependencies. We can circumvent this by linking with -Wl,--as-needed but I'm not sure whether this is a valid workaround

Re: [pacman-dev] [PATCH] Fix handling of ignored packages

2011-03-19 Thread Xavier Chantry
-by: Pang Yan Han pangyan...@gmail.com --- Signed-off-by: Xavier Chantry chantry.xav...@gmail.com

Re: [pacman-dev] libalpm linked against libarchive/libfetch deps (was: SyncFirst and dependencies)

2011-03-19 Thread Xavier Chantry
On Fri, Mar 18, 2011 at 3:18 PM, Lukas Fleischer archli...@cryptocrack.de wrote: Well, it actually is a bit more than just a wrong dependency. ioni and me figured that libalpm 6.0.0 is linked against all libfetch and libarchive dependencies as well whereas 5.0.3 isn't, so there must have been

Re: [pacman-dev] libalpm linked against libarchive/libfetch deps (was: SyncFirst and dependencies)

2011-03-19 Thread Xavier Chantry
On Sat, Mar 19, 2011 at 10:36 AM, Xavier Chantry chantry.xav...@gmail.com wrote: On Fri, Mar 18, 2011 at 3:18 PM, Lukas Fleischer archli...@cryptocrack.de wrote: Well, it actually is a bit more than just a wrong dependency. ioni and me figured that libalpm 6.0.0 is linked against all libfetch

Re: [pacman-dev] SyncFirst and dependencies

2011-03-18 Thread Xavier Chantry
On Fri, Mar 18, 2011 at 12:55 PM, Lukas Fleischer archli...@cryptocrack.de wrote: Hi, I'm not really sure if this actually is a bug or intended behaviour but upgrading pacman man my freshly installed system with [testing] enabled just broke pacman: $ pacman pacman: error while loading

[pacman-dev] [PATCH] libalpm/diskspace.c: remove bogus parenthesis

2011-03-01 Thread Xavier Chantry
--- lib/libalpm/diskspace.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/libalpm/diskspace.c b/lib/libalpm/diskspace.c index 253e118..bfc63e7 100644 --- a/lib/libalpm/diskspace.c +++ b/lib/libalpm/diskspace.c @@ -323,7 +323,7 @@ int _alpm_check_diskspace(pmtrans_t

Re: [pacman-dev] [PATCH] Fix double close of the lock file

2011-02-27 Thread Xavier Chantry
(which reused the original file descriptor) was closed instead. Signed-off-by: Jonathan Conder jonno.con...@gmail.com --- So no one ever spotted this because logs are only made during a transaction ? Anyway nice finding :) Signed-off-by: Xavier Chantry chantry.xav...@gmail.com

Re: [pacman-dev] Removing empty directories

2011-02-27 Thread Xavier Chantry
On Sun, Feb 27, 2011 at 1:37 AM, Allan McRae al...@archlinux.org wrote: I just noticed this: pacman -Qk | grep ^warning warning: filesystem: /opt/ (No such file or directory) warning: hal: /usr/share/hal/fdi/information/20thirdparty/ (No such file or directory) warning: hal-info:

Re: [pacman-dev] Removing empty directories

2011-02-27 Thread Xavier Chantry
On Sun, Feb 27, 2011 at 11:53 AM, Allan McRae al...@archlinux.org wrote: I had a fair suspicion this had been discussed before...  I just did a wtf? when I saw /opt had disappeared. I'm going to create a pactest for this. If we believe the performance hit to fix this problem is not worth

Re: [pacman-dev] Removing empty directories

2011-02-27 Thread Xavier Chantry
On Sun, Feb 27, 2011 at 12:40 PM, Allan McRae al...@archlinux.org wrote: How much of a performance hit would there really be?   If I have this right in my head (big assumption), the file-lists for all packages are read in in one go.  So all that we would have to do is loop over them checking

Re: [pacman-dev] Style questions and concerns

2011-02-26 Thread Xavier Chantry
On Fri, Feb 25, 2011 at 5:27 PM, Dan McGee dpmc...@gmail.com wrote: I've noticed these things a lot in recent patches, so let the discussion commence. 1) typedef-ed structs. This is just a copy the rest of them habit, but I really feel we should stop doing this, only typedef-ing when we are

Re: [pacman-dev] [PATCH 6/6] New VerbosePkgLists option

2011-02-26 Thread Xavier Chantry
On Mon, Feb 21, 2011 at 8:02 PM, Jakob Gruber jakob.gru...@gmail.com wrote: If enabled, displays package lists for upgrade, sync and remove operations formatted as a table. Falls back to default list display if insufficient terminal columns are available. Example output (-Su): Targets (25):

Re: [pacman-dev] License for new contributions?

2011-02-26 Thread Xavier Chantry
On Sat, Feb 26, 2011 at 6:42 PM, edmeiste...@hushmail.com wrote: Hello pacman team! I've been following development for quite some time, and would like to submit my package signing patches for review. Out of curiosity, what do these patches accomplish exactly ? However, since some of the

Re: [pacman-dev] DRAFT: API changes for pacman-3.5

2011-02-19 Thread Xavier Chantry
On Mon, Feb 7, 2011 at 4:58 AM, Allan McRae al...@archlinux.org wrote: This is a draft for the README file update for pacman-3.5. I have flagged two areas that I am not particularly clear at what happened.  I would be great if the people involved in those changes could make the appropriate

Re: [pacman-dev] [ Package Signing ] Your signature please

2011-02-19 Thread Xavier Chantry
On Sat, Feb 19, 2011 at 2:06 PM, IgnorantGuru jgj7.pacman...@mailnull.com wrote: Interesting that you think so, because patches are the way to make non-secure junk.  The way to make things work is for the person most familiar with the code and protocols to make those changes rather than him

Re: [pacman-dev] Version specs in replaces

2011-02-18 Thread Xavier Chantry
On Fri, Feb 18, 2011 at 9:08 PM, Dan McGee dpmc...@gmail.com wrote: On Thursday, February 17, 2011, Daenyth Blank daenyth+a...@gmail.com wrote: On Thu, Feb 17, 2011 at 06:00, Allan McRae al...@archlinux.org wrote: On 17/02/11 10:42, Dan McGee wrote: So the first command makes sense, and at

Re: [pacman-dev] 3.5.0 release planning

2011-02-01 Thread Xavier Chantry
On Tue, Feb 1, 2011 at 7:36 AM, Xyne x...@archlinux.ca wrote: Dan McGee wrote: * Allan's pkghash branch and work. This should be good to go, we were just sorting out a few lingering issues. Does that include the move to parsing sync database archives without extraction? No. This was a

Re: [pacman-dev] sodeps: makepkg patches

2011-02-01 Thread Xavier Chantry
On Tue, Feb 1, 2011 at 3:53 AM, Dan McGee dpmc...@gmail.com wrote: I have the feeling this has been hashed/rehashed/blended before, but why aren't we just doing something like:    provides=('myenv')    libprovides=('libx.so' 'liby.so') To me, this seems a lot clearer and alleviates the

Re: [pacman-dev] [PATCH] Remove need to explicitly register the local DB

2011-01-29 Thread Xavier Chantry
On Sat, Jan 29, 2011 at 10:09 AM, Xyne x...@archlinux.ca wrote: Even if it is negligible, it's munging separate logic together. Given that the purpose is presumably to save coders a few lines of code, I still think it would make more sense to use a wrapper function to do that when getting the

Re: [pacman-dev] Errors while KDE 4.6 upgrading

2011-01-27 Thread Xavier
Sergej Pupykin wrote: Hi, Why pacman think that file is corrupted when I do -Syu and does not think so when I do -U ... ? because the checksum is stored in the Sync database, which -U has no knowledge of. -U only knows the package file you give it. $ pacman -Syu . :: File

[pacman-dev] [PATCH] Makefile: Use git describe --dirty for GIT VERSION

2011-01-22 Thread Xavier Chantry
dirty indicates if the repo has uncommited changes or not when building, so dont hardcode this info. Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- doc/Makefile.am|2 +- scripts/Makefile.am|2 +- src/pacman/Makefile.am |2 +- 3 files changed, 3 insertions

Re: [pacman-dev] [PATCH] Add interactive provider selection

2011-01-22 Thread Xavier Chantry
On Fri, Nov 5, 2010 at 4:55 AM, Allan McRae al...@archlinux.org wrote: I think that if we are providing a selection dialog for provides then it should present packages from all repos to choose from.  Otherwise it seems a half feature to me. So my working branch provides a full feature now !

Re: [pacman-dev] [PATCH] Add interactive provider selection

2011-01-22 Thread Xavier Chantry
On Sun, Jan 23, 2011 at 2:39 AM, Allan McRae al...@archlinux.org wrote: Awesome!   Incredibly minor and pedantic points...  The blank line between the final line of providers and Which one do you want to install? is not there when selecting members of a group: e.g.   53) udev  54) usbutils

Re: [pacman-dev] when upgrading display versions

2011-01-20 Thread Xavier Chantry
On Thu, Jan 20, 2011 at 8:17 PM, Divan Santana di...@s-tainment.co.za wrote: On Thursday 20 January 2011 19:39:18 Dan McGee wrote: You mean like this? https://bugs.archlinux.org/task/15772 Darn it... I should have seen that! I obviously didn't search thorough enough. I have always wanted

Re: [pacman-dev] [PATCH 2/5] alpm/depcmp: new NODEPVERSION flag

2011-01-19 Thread Xavier Chantry
On Wed, Jan 19, 2011 at 7:57 PM, Dan McGee dpmc...@gmail.com wrote: On Wed, Jan 19, 2011 at 11:13 AM, Florian Pritz bluew...@xssn.at wrote: From: Xavier Chantry chantry.xav...@gmail.com This flag allows to disable version checking in dependency resolving code. depcmp_tolerant respects

Re: [pacman-dev] [GIT] The official pacman repository annotated tag, v3.4.2, created. v3.4.2

2010-12-30 Thread Xavier
Dan McGee wrote: 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 annotated tag, v3.4.2 has been created at

Re: [pacman-dev] [GIT] The official pacman repository annotated tag, v3.4.2, created. v3.4.2

2010-12-30 Thread Xavier Chantry
On Thu, Dec 30, 2010 at 4:20 PM, Dan McGee dpmc...@gmail.com wrote: Speaking up about this now? I thought I've asked everyone if there was more for 3.4.2 at least a few times in the last few weeks... I'll take a look at Jakob's patches for maint. I thought I did but I couldn't find any

Re: [pacman-dev] [PATCH] Declare all local functions static

2010-12-19 Thread Xavier Chantry
On Sun, Dec 19, 2010 at 2:31 AM, Allan McRae al...@archlinux.org wrote: be_local.c:70:13: warning: no previous prototype for '_cache_get_filename' [-Wmissing-prototypes] -Wmissing-prototypes aims to detect global functions that fail to be declared in header files.  Given we probably do not

Re: [pacman-dev] [PATCH 2/2] Overhaul archive fgets function

2010-12-19 Thread Xavier Chantry
On Wed, Dec 15, 2010 at 7:48 AM, Dan McGee d...@archlinux.org wrote: The old function was written in a time before we relied on it for nearly every operation. Since then, we have switched to the archive backend and now fast parsing is a big deal. The former function made a per-character call

Re: [pacman-dev] [PATCH 0/4] Package list find performance improvements

2010-12-14 Thread Xavier Chantry
On Tue, Dec 14, 2010 at 7:46 PM, Dan McGee d...@archlinux.org wrote: This series of patches makes finding a package in our linked list implementation a whole lot faster, if that search is using the standard _alpm_pkg_find, which nearly all are (after the first patch). It does this by adding a

Re: [pacman-dev] Warning to pacman-git users - database format change

2010-12-13 Thread Xavier Chantry
On Mon, Dec 13, 2010 at 5:19 AM, Allan McRae al...@archlinux.org wrote: Pacman's local database format changes with this commit being pushed to master: commit 21833d90e26635fdd2c6af247790a9bf374b4d80 Author: Allan McRae al...@archlinux.org Date:   Sat Oct 30 15:35:43 2010 +1000    Merge

Re: [pacman-dev] Misleading info when epoch is used

2010-12-08 Thread Xavier Chantry
On Tue, Dec 7, 2010 at 11:44 PM, Dan McGee dpmc...@gmail.com wrote: On Tue, Dec 7, 2010 at 4:55 PM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: In fact I don't like neither force nor epoch. Epoch is just a version prefix, why don't we let the packager to workaround this (KISS)? We can introduce a

Re: [pacman-dev] Disk space checking branch complete

2010-11-17 Thread Xavier Chantry
On Wed, Nov 17, 2010 at 1:06 AM, Dan McGee dpmc...@gmail.com wrote: We've putzed around with this a few times, haven't we? This basically reverts this one: commit 149839c5391e9a93465f86dbb8d095a0150d755d Author: Xavier Chantry shinin...@gmail.com Date:   Mon May 26 23:46:01 2008 +0200

Re: [pacman-dev] [PATCH] Warn when synchronizing without upgrading (-Sy)

2010-11-17 Thread Xavier Chantry
On Wed, Nov 17, 2010 at 10:00 PM, Thomas Bahn thomas-b...@gmx.net wrote: this patch introduces a question on every repository sync if a am right. So that patch also asks if you type 'pacman -Syu' ? If it is i would not vote for this patch, because it's another question you must acknoweledge

Re: [pacman-dev] Versioned packages on the command line.

2010-11-11 Thread Xavier Chantry
On Thu, Nov 11, 2010 at 8:46 PM, Xyne x...@archlinux.ca wrote: Hi, If two repos (obviously not both official) provide the same binary package, pacman will install the package from the repo that is listed first in pacman.conf, if specified on the command line, e.g. pacman -S foo. If another

Re: [pacman-dev] [RFC] New member selection when installing group

2010-11-04 Thread Xavier Chantry
On Thu, Nov 4, 2010 at 10:53 AM, Allan McRae al...@archlinux.org wrote: My general impression of this is that it is really, really good and a great improvement over what we have now or had in the past.   Here are relatively minor comments: The Enter a number selection prompt is a bit weirdly

Re: [pacman-dev] [PATCH] Add interactive provider selection

2010-11-04 Thread Xavier Chantry
On Thu, Nov 4, 2010 at 11:00 AM, Allan McRae al...@archlinux.org wrote: On 20/10/10 05:10, Xavier Chantry wrote: If there are multiple providers in one db, pacman used to just stop at the first one (both during dependency resolution or for pacman -S 'provision' which uses the same code). I

Re: [pacman-dev] some pacman work

2010-11-01 Thread Xavier Chantry
On Wed, Oct 20, 2010 at 6:54 AM, Allan McRae al...@archlinux.org wrote: pactree: all the work I did with Dave Reisner on pactree C rewrite, all ready and done now. depends on depwork for find_satisfier Cool.  Still not sure about moving it to util instead of contrib.  It does not really fit

Re: [pacman-dev] [PATCH] Add function to list mount points

2010-10-31 Thread Xavier Chantry
comments by Xavier is on my working branch (http://projects.archlinux.org/users/allan/pacman.git/log/?h=working). Allan If we agree on the #if defined .. #elif defined .. #endif , we should probably do it for the includes too.

Re: [pacman-dev] [PATCH] Sync data after writing to the local db

2010-10-30 Thread Xavier Chantry
On Sat, Oct 30, 2010 at 3:01 PM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: Wouldn't this cause a notable slow-down? I thought the issue was rather that the local db entries would be synced when the actual package files are probably not synced. But if we sync everything, then it's very likely to

Re: [pacman-dev] epoch (was: Chat with toofis...@jabber.org)

2010-10-30 Thread Xavier Chantry
On Mon, Oct 25, 2010 at 1:03 PM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: Nagy and I discussed a bit that topic. Don't we already read all local depends file for conflict and/or dep checking ? So this means we'll now read all local depends + all desc files ? Why not put epoch stuff in the

Re: [pacman-dev] epoch

2010-10-30 Thread Xavier Chantry
On Sat, Oct 30, 2010 at 11:26 PM, Allan McRae al...@archlinux.org wrote: That is because none of the packages know about epoch yet.   You are going to have to manually update packages that use the force flag for the time being. BTW, the db update in [testing] has epoch=1 as the db-4.9

Re: [pacman-dev] [PATCH] Add interactive provider selection

2010-10-20 Thread Xavier
Xyne wrote: Is the install size shown after each package when ShowSize is enabled? That would be useful when deciding which package to install. It would be even more useful to know the cumulative size, i.e. the sum of install sizes of that package and its own dependencies. I think that would

Re: [pacman-dev] [RFC] New member selection when installing group

2010-10-20 Thread Xavier
Florian Pritz wrote: On 19.10.2010 21:25, Xavier Chantry wrote: But my new provision selection inspired to both Jakob and Dan that we could have the same for groups, so I went ahead and implemented it over the weekend. Do it like yaourt (it prompts the user when doing -Syu) and open

Re: [pacman-dev] [RFC] New member selection when installing group

2010-10-20 Thread Xavier Chantry
On Wed, Oct 20, 2010 at 8:27 PM, Florian Pritz bluew...@server-speed.net wrote: Yaourt doesn't have this feature for groups (sadly), but for -Syu. You can use the editor to select which packages you want to upgrade and with ones you want to ignore. I just meant you can do the same thing for

Re: [pacman-dev] some pacman work

2010-10-19 Thread Xavier Chantry
About my pending work on http://code.toofishes.net/cgit/xavier/pacman.git/ parseargs: sent to the ML and updated according to feedback depwork: forgot if I send this stuff, should I ? pactree: all the work I did with Dave Reisner on pactree C rewrite, all ready and done now. depends on depwork

[pacman-dev] [PATCH] Add interactive provider selection

2010-10-19 Thread Xavier Chantry
) courier-mta 2) esmtp 3) exim Which one do you want to install? Enter a number (default=1): Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- lib/libalpm/alpm.h|1 + lib/libalpm/deps.c| 28 +- src/pacman/callback.c | 13 ++ src/pacman/util.c

Re: [pacman-dev] Chat with toofis...@jabber.org

2010-10-15 Thread Xavier Chantry
On Fri, Oct 8, 2010 at 7:06 PM, toofis...@jabber.org toofis...@jabber.orgwrote: 5:48 PM me: waoh, where does this come from ? :) what made you implement epoch ? toofishes: the stupid bug reports clicked on one and went ok we have three of these lets fix this shit 5:49 PM it ended up not

[pacman-dev] [PATCH] Sort short options before long options in --help

2010-10-14 Thread Xavier Chantry
the operation --print-format string specify how the targets should be printed Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- src/pacman/pacman.c | 27 ++- 1 files changed, 26 insertions(+), 1 deletions(-) diff --git a/src/pacman/pacman.c

[pacman-dev] [PATCH 1/3] add 4 pactests for -Sdd

2010-10-14 Thread Xavier Chantry
Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- test/pacman/tests/sync-sdd1.py | 18 ++ test/pacman/tests/sync-sdd2.py | 15 +++ test/pacman/tests/sync-sdd3.py | 15 +++ test/pacman/tests/sync-sdd4.py | 17 + 4 files

[pacman-dev] [PATCH 3/3] add -dd option

2010-10-14 Thread Xavier Chantry
From: Florian Pritz bluew...@xssn.at -dd ignores only the version of a dependency being checked, but not the package itself. Signed-off-by: Florian Pritz bluew...@xssn.at Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- doc/pacman.8.txt |1 + src/pacman/pacman.c

Re: [pacman-dev] [PATCH] Sort short options before long options in --help

2010-10-14 Thread Xavier Chantry
On Thu, Oct 14, 2010 at 1:55 PM, Nagy Gabor ng...@bibl.u-szeged.hu wrote: I like this. Cool :) +     /* First skip all spaces in both strings */ +     while(isspace((unsigned char)*s1)) +             s1++; +     while(isspace((unsigned char)*s2)) +             s2++; +     /* If we

[pacman-dev] [PATCH] Sort short options before long options in --help

2010-10-14 Thread Xavier Chantry
the operation --print-format string specify how the targets should be printed Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- src/pacman/pacman.c | 39 ++- 1 files changed, 38 insertions(+), 1 deletions(-) diff --git a/src

Re: [pacman-dev] [PATCH] Sort short options before long options in --help

2010-10-14 Thread Xavier Chantry
that you do not want to keep in a release build, but you want to make sure that the assert is indeed not triggered while developing / testing. I updated the patch with an explicit check (I actually already did this before choosing to use assert) : http://code.toofishes.net/cgit/xavier/pacman.git/log/?h

Re: [pacman-dev] [PATCH] Read 'force' entry from packages

2010-10-14 Thread Xavier Chantry
On Tue, Oct 12, 2010 at 12:51 AM, Dan McGee d...@archlinux.org wrote: We weren't reading this in from our packages, thus causing us not to write it out to our local database. Adding this now will help ease the upgrade path for epoch later and not require reinstallation of all force packages.

Re: [pacman-dev] [PATCH] Read 'force' entry from packages

2010-10-14 Thread Xavier Chantry
On Thu, Oct 14, 2010 at 8:48 PM, Xavier Chantry chantry.xav...@gmail.com wrote: On Tue, Oct 12, 2010 at 12:51 AM, Dan McGee d...@archlinux.org wrote: We weren't reading this in from our packages, thus causing us not to write it out to our local database. Adding this now will help ease

Re: [pacman-dev] [GIT] The official pacman repository branch, master, updated. v3.4.1-88-g46ffd34

2010-10-13 Thread Xavier Chantry
On Tue, Oct 12, 2010 at 12:38 AM, Dan McGee dpmc...@gmail.com wrote: Xavier and I talked about it, and my numbers were a bit flawed due to having a debug-compiled pacman and comparing it against the system pacman. We also figured it would help a lot on patches with a slow stat (Cygwin, etc

Re: [pacman-dev] [PATCH 4/4] CLI args: update --help and manpage

2010-10-12 Thread Xavier Chantry
On Tue, Oct 12, 2010 at 4:04 AM, Dan McGee dpmc...@gmail.com wrote: I'm not too thrilled abuot this patch for the fact that we lost alpha ordering by shortopt for sync/upgrade/remove options. This is a step backwards in my opinion. I can deal with the dbpath/root/every single op uses these

[pacman-dev] [PATCH] alpm/remove.c : respect --dbonly during remove-upgrade

2010-10-11 Thread Xavier Chantry
the DB entry without touching anything on the file stem. E.g. this test should be the same as: pacman -R --dbonly dummy pacman -U --dbonly dummy.pkg.tar.gz Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- doc/pacman.8.txt |2 -- lib/libalpm/remove.c

[pacman-dev] [PATCH] add 3 pactests for -Sdd

2010-10-10 Thread Xavier Chantry
Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- test/pacman/tests/sync-sdd1.py | 15 +++ test/pacman/tests/sync-sdd2.py | 15 +++ test/pacman/tests/sync-sdd3.py | 15 +++ 3 files changed, 45 insertions(+), 0 deletions(-) create mode 100644

[pacman-dev] [PATCH 3/4] CLI args: stricter/better parsing

2010-10-10 Thread Xavier Chantry
From: Jakob Gruber jakob.gru...@gmail.com In the following, the letters SRUDQ refer to the corresponding pacman operations. Most of the work in this commit is about removing as many options as possible from the global section and moving them to where they actually belong. Additionally,

[pacman-dev] [PATCH 4/4] CLI args: update --help and manpage

2010-10-10 Thread Xavier Chantry
components together. Duplication is also avoided. Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- doc/pacman.8.txt| 110 -- src/pacman/pacman.c | 164 +-- 2 files changed, 133 insertions(+), 141 deletions

[pacman-dev] [PATCH 1/2] pactest: use simpler method to create tar

2010-10-10 Thread Xavier Chantry
just like in 24fc623e1a8bf905cf0367f9bd40bc5bd6034378 , apply to pmdb gensync as well. --- test/pacman/pmdb.py |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/test/pacman/pmdb.py b/test/pacman/pmdb.py index 41bd738..11083db 100755 --- a/test/pacman/pmdb.py +++

[pacman-dev] [PATCH 2/2] pactest: fix gensync

2010-10-10 Thread Xavier Chantry
gensync generated a sync.db file with PKGINFO syntax, this is not quite what pacman expects. Also the file was only added to the Server path : root/var/pub/sync/sync.db but it was not available in the normal sync db path : root/var/lib/pacman/sync/sync.db change gensync() to generate

Re: [pacman-dev] [PATCH 0/5] RFC: epoch implementation

2010-10-09 Thread Xavier Chantry
On Sat, Oct 9, 2010 at 3:47 AM, Allan McRae al...@archlinux.org wrote: I just took these for a spin al...@mugen /home/arch/code/pacman (epoch) sudo ./src/pacman/pacman -Syu snip Targets (24): blas-3.2.2-2  crafty-23.3-1  db-4.8.26-2              foomatic-db-4.0.5_20100816-1

Re: [pacman-dev] Sodeps support

2010-10-08 Thread Xavier Chantry
for -1. Also slightly changed the second patch to do just once : if (nodeps) else http://code.toofishes.net/cgit/xavier/pacman.git/log/?h=sodeps

Re: [pacman-dev] Groups. Again.

2010-10-04 Thread Xavier
Xyne wrote: I'm re-proposing that we scrap groups entirely in favor of metapackages. (I think I send a message to this list about this over a year ago.) A metapackage would consist of an empty package (i.e. no installable files other than pacman metadata). The package would list the

Re: [pacman-dev] Groups. Again.

2010-10-04 Thread Xavier
Allan McRae wrote: One thing I just thought of. If it is going to be separated by repo could that line say :: group testing/base-devel to improve clarity? This is exactly what Jakob reported as inconsistent, but wondered if it was still ok. That's funny, we have been arguing for one

Re: [pacman-dev] Groups. Again.

2010-10-03 Thread Xavier
priority. I was going to cover them here, but this will just confuse things and would deserve its own topic/discussion. Jakob Gruber wrote: Hi Xavier, here is yet another revision of the group patches. http://github.com/schuay/pacman-arch/commits/group_backend This time it should be easier

Re: [pacman-dev] [PATCH] makepkg: canonicalize paths from environmental variables

2010-09-30 Thread Xavier
Well, one way to avoid all this is to add something like: [[ $PKGDIR == . ]] $PKGDIR=$startdir before we preserve the environmental variables. But that is bad as things like PKGDIR=../ would still cause issues. I rather meant a way to remove any assumption of PKGDEST being

Re: [pacman-dev] [PATCH] makepkg: canonicalize paths from environmental variables

2010-09-30 Thread Xavier
Allan McRae wrote: No. It also affects the location of the tarfile. e.g. PKGDEST=. puts the tar file in $pkgdir because we create the tarball location using $PKGDEST while in $pkgdir: cd $pkgdir ... pkg_file=$PKGDEST/${nameofpkg}-${pkgver}-${pkgrel}-${PKGARCH}${PKGEXT} ... ok but

Re: [pacman-dev] [PATCH] makepkg: canonicalize paths from environmental variables

2010-09-29 Thread Xavier Chantry
On Wed, Sep 29, 2010 at 3:52 PM, Allan McRae al...@archlinux.org wrote: Just to clarify (thanks to Xavier), it worked on BSD but not in OSX which does not have either readlink -f or realpath... just found a link that might help : http://stackoverflow.com/questions/1055671/how-can-i-get

Re: [pacman-dev] python-2.7 and pacman test-suite

2010-09-28 Thread Xavier Chantry
On Tue, Sep 28, 2010 at 4:23 PM, Dan McGee dpmc...@gmail.com wrote: Xavier, any insight or memory why you did it this way? commit c465d9e848b19b495259c7021a583c29fba92b44 Author: Chantry Xavier shinin...@gmail.com Date:   Thu Apr 17 09:02:11 2008 +0200    pactest : Use tarfile module

Re: [pacman-dev] [PATCH 1/5] Tests: Adjust sync023 to correctly test FS#20221

2010-09-27 Thread Xavier Chantry
On Mon, Sep 27, 2010 at 8:38 PM, Jakob Gruber jakob.gru...@gmail.com wrote: I couldn't see how the original test made any sense, so I adjusted it to test FS#20221 (as specified in commit e702f56ea671c6cd1154a0ddb41fa63e97587c85). The old PKG_EXIST rules do nothing because pkg{1,2,3} are

Re: [pacman-dev] [PATCH 2/2] use execv to avoid using sh just to run ldconfig

2010-08-18 Thread Xavier Chantry
://code.toofishes.net/cgit/xavier/pacman.git/log/?h=working

Re: [pacman-dev] replaces vs conflicts

2010-08-13 Thread Xavier Chantry
On Fri, Aug 13, 2010 at 1:46 PM, Allan McRae al...@archlinux.org wrote: I got a conflict too. pacman 3.4.0-2 Great, I am not going mad.   But not great, something strange is happening... Can you show debug output ? And libelf is not in any of your sync repo, ie pacman -Si libelf does not

Re: [pacman-dev] replaces vs conflicts

2010-08-13 Thread Xavier Chantry
On Fri, Aug 13, 2010 at 8:54 PM, Florian Pritz bluew...@server-speed.net wrote: On 13.08.2010 20:42, Xavier Chantry wrote: On Fri, Aug 13, 2010 at 1:46 PM, Allan McRae al...@archlinux.org wrote: I got a conflict too. pacman 3.4.0-2 Great, I am not going mad.   But not great, something

Re: [pacman-dev] Explicit (opt)deps for makepkg.

2010-07-19 Thread Xavier Chantry
On Mon, Jul 19, 2010 at 12:15 PM, k...@krzd.net wrote: Sure that in this case you don't need a patch, but where is the problem to file a simple bug report. Just two lines of text and everything is allright. Just my two cents Please... the developers can talk for themselves... and these

Re: [pacman-dev] [PATCH 2/6] Remove use of gpgme

2010-07-08 Thread Xavier Chantry
On Wed, Jul 7, 2010 at 7:43 PM, Denis A. Altoé Falqueto denisfalqu...@gmail.com wrote: gpgme is very good when the keyring is the default, or if the requirements are simple. But for a specialized keyring, like pacman's, it is not flexible enough. This patch removes the checking of configure.ac.

[pacman-dev] [PATCH 3/3] manpage: add real world examples

2010-07-01 Thread Xavier Chantry
From: Nico Schottelius n...@kr.ethz.ch Signed-off-by: Nico Schottelius n...@kr.ethz.ch Signed-off-by: Xavier Chantry chantry.xav...@gmail.com --- doc/pacman.8.txt | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/doc/pacman.8.txt b/doc/pacman.8.txt index

Re: [pacman-dev] [PATCH] makepkg: fix sudo/su calling of pacman

2010-07-01 Thread Xavier Chantry
On Thu, Jul 1, 2010 at 3:08 AM, Loui Chang louipc@gmail.com wrote: On Wed 30 Jun 2010 20:08 -0430, Andres P wrote: Also, how do I unsubscribe from this shitty list? ;) Heyy. Don't be so rude now. The list isn't that bad. http://mailman.archlinux.org/mailman/listinfo/pacman-dev I always

  1   2   3   4   5   6   7   >