On Fri, 11 Nov 2016 at 21:15:48, Christian Hesse wrote:
> From: Christian Hesse
>
> 'YES' translates to 'JA' in German, thus answer 'J' is expected for
> positive answer. This changes the behaviour to always accept 'Y'
> and 'N', in addition to the translated values.
>
>
Add a space between the option and its argument. Also, do not enclose
the argument in asterisk characters.
Fixes a typo introduced in aa4c61f (Document database checking options,
2014-12-28).
Reported-by: Luca Weiss <luca.emanuel.we...@gmail.com>
Signed-off-by: Lukas Fleischer &l
On Tue, 23 Feb 2016 at 20:42:36, Lukas Jirkovsky wrote:
> On 23 February 2016 at 20:20, Gesh wrote:
> > I am confused. Given that h hooks support is only provided by pacman >=5,
> > shouldn't such PKGBUILDs require that in their depends array?
> >
> > Just my two cents,
> >
Use alpm_find_satisfier() instead of alpm_db_get_pkg() when retrieving
the install status of a package to make sure we spot providers as well.
Signed-off-by: Lukas Fleischer lfleisc...@archlinux.org
---
src/pacman/package.c | 3 ++-
src/pacman/util.c| 3 ++-
2 files changed, 4 insertions
On Tue, 26 Aug 2014 at 14:52:10, Dave Reisner wrote:
[...]
Again with the lack of quoting...
(printf $PWD/%s\n *.pkg.tar?(.+([^.])) | pacsort --files |
I'm somewhat concerned about this because we're injecting a path into
the format string (yes, we've done this elsewhere and it's
On Tue, 26 Aug 2014 at 15:11:12, Lukas Fleischer wrote:
On Tue, 26 Aug 2014 at 14:52:10, Dave Reisner wrote:
[...]
Again with the lack of quoting...
(printf $PWD/%s\n *.pkg.tar?(.+([^.])) | pacsort --files |
I'm somewhat concerned about this because we're injecting a path
On Wed, Jun 19, 2013 at 09:14:31PM +1000, Allan McRae wrote:
On 19/06/13 15:23, Andrew Gregory wrote:
Relocated query_group() to allow calling filter().
Fixes FS#19716
Signed-off-by: Andrew Gregory andrew.gregor...@gmail.com
Signed-off-by: Me
Aside:
I have just noticed how
On Sun, Mar 24, 2013 at 11:12:05AM -0700, Anatol Pomozov wrote:
Hi,
I believe in automatization. Any routine work that can be done
automatically should be done this way.
One such thing that can be improved in Arch project is discovering
out-of-date packages. Currently it is done by users
On Sun, Mar 24, 2013 at 11:26:04AM -0700, Anatol Pomozov wrote:
Hi
On Sun, Mar 24, 2013 at 10:57 AM, Anatol Pomozov
anatol.pomo...@gmail.com wrote:
Hi,
I am an update junkie. I do pacman -Suy several times a day. One
thing I do not like with current pacman is that it redownloads whole
On Tue, Jul 17, 2012 at 03:23:12AM -0300, Martin Cigorraga wrote:
Hi guys!
I was thinking it could be a cool feature if pacman can warn the user every
time news are published on our main page by issuing an alert when:
1. issuing a pacman -Syu -- which involves full system upgrade
2.
On Tue, Jul 17, 2012 at 11:50:26AM +1000, Allan McRae wrote:
Detect a conflict between a file/symlink in one package and a directory
in another when both are being installed at once.
A side effect is the creation on conflicts between a direcotry symlink
and a real directory (e.g lib -
On Fri, Mar 09, 2012 at 11:20:03PM +1000, Allan McRae wrote:
Ignore .sig, *.db*, and *.src.tar* when cleaning the package cache.
Fixes FS#25166.
Signed-off-by: Allan McRae al...@archlinux.org
---
I am bad at regex... so I am sure some of these could be tightened up.
Why use regular
On Wed, Feb 15, 2012 at 07:34:54AM +0100, Pierre Schmitz wrote:
Am 14.02.2012 23:08, schrieb Dave Reisner:
On Tue, Feb 14, 2012 at 02:30:06PM -0600, Dan McGee wrote:
This has outlived its usefulness and causes more problems than it
solves.
Signed-off-by: Dan McGee d...@archlinux.org
On Thu, Dec 22, 2011 at 04:55:12PM +1000, Allan McRae wrote:
Using the value of SKIP in the checksum array will cause that
integrity check to be skipped. This makes building packages that
rely on user configurable sources less painful.
Based-on-patch-by: Dan McGee d...@archlinux.org
source directory. This seems like the most natural approach.
We install those separately and should act the same when building
stuff.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
Yes, I'm starting to become pushy! :) Seriously - this should do what we
discussed earlier. Dan, you can
with exit status 1 if we display the usage message due to
invalid arguments.
* Add AUTOMAKE_OPTIONS = std-options and add all scripts to
bin_SCRIPTS to make `make installcheck` check that installed scripts
actually support the --help and --version options.
Signed-off-by: Lukas Fleischer archli
On Sat, Dec 10, 2011 at 09:28:03PM +1000, Allan McRae wrote:
On 08/12/11 02:49, Lukas Fleischer wrote:
On Tue, Dec 06, 2011 at 05:05:20PM -0600, Dan McGee wrote:
On Tue, Dec 6, 2011 at 4:58 PM, Lukas Fleischer
archli...@cryptocrack.de wrote:
On Tue, Dec 06, 2011 at 04:38:41PM -0600, Dan
On Tue, Dec 06, 2011 at 05:05:20PM -0600, Dan McGee wrote:
On Tue, Dec 6, 2011 at 4:58 PM, Lukas Fleischer
archli...@cryptocrack.de wrote:
On Tue, Dec 06, 2011 at 04:38:41PM -0600, Dan McGee wrote:
On Tue, Dec 6, 2011 at 3:29 PM, Lukas Fleischer
archli...@cryptocrack.de wrote:
Can some
Add a read-only variable $myname to every contrib script and hardcode
program names instead of relying on $0. The variable name $myname
was chosen because it is already used in pacman and because we use
$myver to specify the program version in the official scripts.
Signed-off-by: Lukas Fleischer
with exit status 1 if we display the usage message due to
invalid arguments.
* Add AUTOMAKE_OPTIONS = std-options and add all scripts to
bin_SCRIPTS to make `make installcheck` check that installed scripts
actually support the --help and --version options.
Signed-off-by: Lukas Fleischer archli
Be more semantically accurate and avoid accidental overwriting of some
configuration variables that are considered to be constant.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
scripts/makepkg.sh.in |8
scripts/pacman-db-upgrade.sh.in |2 +-
scripts
On Sat, Dec 03, 2011 at 11:56:09AM -0600, Dan McGee wrote:
On Fri, Dec 2, 2011 at 6:44 PM, Lukas Fleischer
archli...@cryptocrack.de wrote:
On Fri, Dec 02, 2011 at 10:10:14AM -0600, Dan McGee wrote:
On Fri, Dec 2, 2011 at 2:49 AM, Lukas Fleischer
archli...@cryptocrack.de wrote
Be consistent with all other contrib scripts and support the --help
command line switch. Fixes FS#27258.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
contrib/paclist.in |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/contrib/paclist.in b/contrib
On Fri, Dec 02, 2011 at 10:10:14AM -0600, Dan McGee wrote:
On Fri, Dec 2, 2011 at 2:49 AM, Lukas Fleischer
archli...@cryptocrack.de wrote:
Be consistent with all other contrib scripts and support the --help
command line switch. Fixes FS#27258.
Where are we on consistency here with what we
(whitespace followed by
a comment is already trimmed by `sed s/[[:space:]]*#.*//`).
Apart from that:
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
scripts/makepkg.sh.in | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/scripts/makepkg.sh.in b/scripts
instead
of occured). Francois already fixed this for makepkg in 1e51b81c.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
scripts/makepkg.sh.in |4 +++-
scripts/repo-add.sh.in |6 --
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/scripts/makepkg.sh.in b
On Wed, Oct 12, 2011 at 08:27:53AM -0500, Dan McGee wrote:
On Tue, Oct 11, 2011 at 1:33 PM, Lukas Fleischer
archli...@cryptocrack.de wrote:
On Tue, Oct 11, 2011 at 07:03:13PM +0200, Lukas Fleischer wrote:
On Tue, Oct 11, 2011 at 11:29:31AM -0500, Dan McGee wrote:
On Tue, Oct 11, 2011
Instead of always showing == ERROR: TERM signal caught. Exiting...,
replace TERM with whatever signal is actually caught.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
scripts/repo-add.sh.in |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/scripts/repo
On Tue, Oct 11, 2011 at 11:29:31AM -0500, Dan McGee wrote:
On Tue, Oct 11, 2011 at 11:13 AM, Lukas Fleischer
archli...@cryptocrack.de wrote:
There's no need to keep the lock file if we interrupt repo-{add,remove}
using ^C (same applies to other signals).
Definite +1 here.
In contrast
to refactor and/or
replace the standalone configuration file parser.
We do not define a short option for the sake of consistency with
pacman's set of command line options.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
src/util/pactree.c | 18 +++---
1 files changed, 15
could not be read
Also, move the error message inside register_syncs() to allow for
differentiating between different errors that might require a handler in
the future.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
src/util/pactree.c |2 +-
1 files changed, 1 insertions(+), 1
On Wed, Sep 21, 2011 at 01:40:41AM -0400, Eric Bélanger wrote:
On Tue, Sep 20, 2011 at 11:40 PM, Allan McRae al...@archlinux.org wrote:
On 21/09/11 09:49, Dan McGee wrote:
On Tue, Sep 20, 2011 at 9:11 AM, Florian Pritzbluew...@xinu.at wrote:
This might help to locate a failing command
On Tue, Sep 13, 2011 at 07:48:18AM -0400, Dave Reisner wrote:
On Tue, Sep 13, 2011 at 01:33:08PM +0200, Lukas Fleischer wrote:
On Tue, Sep 13, 2011 at 07:25:29AM -0400, Dave Reisner wrote:
On Tue, Sep 13, 2011 at 07:00:27AM -0400, Dan McGee wrote:
On Tue, Sep 13, 2011 at 6:54 AM, Sergej
config_set_arch() already calls strdup(). Remove strdup() from the
config_set_arch() invocation to avoid a memory leak.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
src/pacman/pacman.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/pacman/pacman.c b
Free syncpath and restore umask if we fail to grab a lock.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
lib/libalpm/be_sync.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/lib/libalpm/be_sync.c b/lib/libalpm/be_sync.c
index 6e64e5f..5e53798 100644
On Thu, Aug 18, 2011 at 07:37:24AM -0500, Dan McGee wrote:
On Thu, Aug 18, 2011 at 2:13 AM, Lukas Fleischer
archli...@cryptocrack.de wrote:
On Wed, Aug 17, 2011 at 04:54:56PM +0200, Lukas Fleischer wrote:
Avoid invoking unlink() with a NULL path if the URL isn't to a file and
we fail
:)
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
lib/libalpm/dload.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c
index 5464740..3798937 100644
--- a/lib/libalpm/dload.c
+++ b/lib/libalpm/dload.c
@@ -396,7
stat()'s behaviour is undefined if the first argument is NULL and might
be prone to segfault. Add an additional check to skip the stat()
invocation if no destfile is used.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
lib/libalpm/dload.c |2 +-
1 files changed, 1 insertions
Avoid a potential segfault that may occur if we use a temporary file and
fail to build the destination file name from the effective URL.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
Another approach is to use a random default destfile name. Since the URL
parsing is very unlikely
On Wed, Aug 17, 2011 at 08:37:23AM -0400, Dave Reisner wrote:
On Wed, Aug 17, 2011 at 10:15:16AM +0200, Lukas Fleischer wrote:
stat()'s behaviour is undefined if the first argument is NULL and might
be prone to segfault. Add an additional check to skip the stat()
invocation if no destfile
On Wed, Aug 17, 2011 at 09:40:22AM -0400, Dave Reisner wrote:
On Wed, Aug 17, 2011 at 10:15:17AM +0200, Lukas Fleischer wrote:
Avoid a potential segfault that may occur if we use a temporary file and
fail to build the destination file name from the effective URL.
Signed-off-by: Lukas
Avoid a potential segfault that may occur if we use a temporary file and
fail to build the destination file name from the effective URL.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
Untested, sorry. Should be trivial enough not to break anything though.
Hopefully.
lib/libalpm
Avoid invoking unlink() with a NULL path if the URL isn't to a file and
we fail to create a temporary file.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
lib/libalpm/dload.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/lib/libalpm/dload.c b/lib
On Tue, Aug 09, 2011 at 12:56:41PM +0200, Sebastien Luttringer wrote:
Before this, cleaning is done when script exit with a value != 0.
If a build fail, directory remain unclean. The purpose of cleaning should
not be changed if build fail.
Signed-off-by: Sebastien Luttringer se...@seblu.net
Immediately jump to the cleanup code after setting the return code to -1
in case rename() fails. Otherwise, it will be reset to 0 right after we
leave the if branch.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
Just spotted this one in maint. I realized the download_internal() code
himself in a situation dire enough to need this.
Signed-off-by: Dave Reisner dreis...@archlinux.org
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
The bracket in the regular expression in line 42 needs to be escaped, otherwise
OpenBSD's awk(1) fails with:
awk: illegal primary
On Sun, Apr 24, 2011 at 11:25:52PM +1000, Allan McRae wrote:
On 23/04/11 09:32, Dan McGee wrote:
Allan and I today, and Denis in the past, noticed some issues with
having a shared public key database and locking that gpg wants to do
when reading from it. Here is an interesting bit from the apt
Drawing progress bars before calling curl_easy_perform() is needless as
the curl progress callback is called with zero progress before actually
downloading the file anyways. Fixes display of 0% progress bars when
sync'ing package databases that are already up to date.
Signed-off-by: Lukas
On Mon, Mar 21, 2011 at 03:01:13PM +1000, Allan McRae wrote:
On 20/03/11 18:08, Allan McRae wrote:
FS#23322 - zsh autocompletion no longer works with pacman 3.5
- The zsh autocompletion file has not been updated in a while.
Do we have a zsh user that wants to tackle this?
I have attempted
On Mon, Mar 21, 2011 at 09:24:35AM -0400, Dave Reisner wrote:
On Mon, Mar 21, 2011 at 07:50:37AM -0500, Dan McGee wrote:
On Mon, Mar 21, 2011 at 5:53 AM, Lukas Fleischer
archli...@cryptocrack.de wrote:
Drawing progress bars before calling curl_easy_perform() is needless as
the curl
On Sun, Mar 20, 2011 at 11:01:27AM -0500, Dan McGee wrote:
On Sat, Mar 19, 2011 at 6:29 PM, Allan McRae al...@archlinux.org wrote:
On 20/03/11 06:28, Lukas Fleischer wrote:
Also includes a Debian patch (from #347650) that makes libtool play
nicely with -Wl,--as-needed.
Signed-off
On Sun, Mar 20, 2011 at 11:29:32AM -0500, Dan McGee wrote:
Applied this but still seeing problems...hmm.
/bin/sh ../../libtool --tag=CC --mode=link gcc -std=gnu99 -pedantic
-D_GNU_SOURCE -fvisibility=internal -fgnu89-inline -march=x86-64
-mtune=generic -O2 -pipe -fstack-protector-all
On Sat, Mar 19, 2011 at 11:38:13AM +0100, Xavier Chantry wrote:
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
Also includes a Debian patch (from #347650) that makes libtool play
nicely with -Wl,--as-needed.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
lib/libalpm/Makefile.am |2 +-
ltmain.sh | 14 ++
2 files changed, 15 insertions(+), 1 deletions(-)
diff
On Sat, Mar 19, 2011 at 09:28:34PM +0100, Lukas Fleischer wrote:
Also includes a Debian patch (from #347650) that makes libtool play
nicely with -Wl,--as-needed.
Signed-off-by: Lukas Fleischer archli...@cryptocrack.de
---
lib/libalpm/Makefile.am |2 +-
ltmain.sh | 14
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 shared libraries: liblzma.so.5: cannot open shared
object file: No such file or directory
On Fri, Mar 18, 2011 at 01:10:01PM +0100, Xavier Chantry wrote:
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
On Fri, Mar 18, 2011 at 02:02:06PM +0100, Lukas Fleischer wrote:
On Fri, Mar 18, 2011 at 01:10:01PM +0100, Xavier Chantry wrote:
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
On Sun, Feb 13, 2011 at 11:42:38AM +0100, Pierre Schmitz wrote:
On Sun, 13 Feb 2011 11:22:54 +0100, Dieter Plaetinck wrote:
with pacman -Qi or pacman -Si we show the person who packaged it, but
not the maintainer.
This makes it harder to get in touch with the right person then it should
On Wed, Sep 29, 2010 at 11:15:45PM +1000, Allan McRae wrote:
Could any BSD/OSX users report on whether realpath . give the full path?
Yes, gives the full path on FreeBSD 8.1-RELEASE. Doesn't work on OpenBSD
tho (zsh: command not found: realpath).
---
doc/PKGBUILD-example.txt |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/doc/PKGBUILD-example.txt b/doc/PKGBUILD-example.txt
index 7245228..99fd8c2 100644
--- a/doc/PKGBUILD-example.txt
+++ b/doc/PKGBUILD-example.txt
@@ -15,6 +15,6 @@
61 matches
Mail list logo