[pacman-dev] [PATCH] libmakepkg: only output package file names when checking for build references

2017-10-08 Thread Allan McRae
The leading directory is distraction from identifying the files containing build references. Signed-off-by: Allan McRae --- scripts/libmakepkg/lint_package/build_references.sh.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [pacman-dev] [PATCH] makepkg: tell the compiler to record debugging info for debug packages

2017-10-08 Thread Allan McRae
On 08/10/17 16:38, Eli Schwartz wrote: > In commit 8b0d59b83a60eb504567590346119fe4cd891cad support was added for > storing the source files of binaries in debug packages. This made use of > the debugedit program which is part of the RPM package manager, which is > not very standalone. > > The

Re: [pacman-dev] [PATCH v2 1/3] makepkg: refactor archive compression for reusability

2017-10-08 Thread Allan McRae
On 08/10/17 16:50, Eli Schwartz wrote: > This allows for more easily extending the list of allowed compression > methods, as it has to be modified in only one place. > > Also allow the user to specify their own preferred command + options for > source packages in addition to compiled packages.

Re: [pacman-dev] [PATCH v2 2/3] makepkg: add support for the zst format

2017-10-08 Thread Allan McRae
On 08/10/17 16:50, Eli Schwartz wrote: > Signed-off-by: Eli Schwartz > --- > doc/makepkg.conf.5.txt | 3 ++- > etc/makepkg.conf.in| 1 + > scripts/libmakepkg/util/compress.sh.in | 1 + > 3 files changed, 4 insertions(+), 1 deletion(-)

Re: [pacman-dev] [PATCH v2 3/3] repo-add: add support for the zst format

2017-10-08 Thread Allan McRae
On 08/10/17 16:50, Eli Schwartz wrote: > Signed-off-by: Eli Schwartz > --- > > I'm now wondering if it makes sense to import > libmakepkg/util/compress.sh here for reuse. Currently repo-add checks > the extensions twice and aborts on unknown extensions, but I'm not sure

[pacman-dev] [PATCH] makepkg: refactor checking for write permissions into a utility function

2017-10-08 Thread Eli Schwartz
Additionally provide a separate error for the confusing if unlikely event that the user tries to use an existing file as a package output directory. This also means we now consistently try to create any nonexistent *DEST directories as needed before aborting with E_FS_PERMISSIONS. Previously only

[pacman-dev] [PATCH v2 2/3] makepkg: add support for the zst format

2017-10-08 Thread Eli Schwartz
Signed-off-by: Eli Schwartz --- doc/makepkg.conf.5.txt | 3 ++- etc/makepkg.conf.in| 1 + scripts/libmakepkg/util/compress.sh.in | 1 + 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/doc/makepkg.conf.5.txt

[pacman-dev] [PATCH v2 3/3] repo-add: add support for the zst format

2017-10-08 Thread Eli Schwartz
Signed-off-by: Eli Schwartz --- I'm now wondering if it makes sense to import libmakepkg/util/compress.sh here for reuse. Currently repo-add checks the extensions twice and aborts on unknown extensions, but I'm not sure why we need to be stricter here than in makepkg

[pacman-dev] [PATCH v2 1/3] makepkg: refactor archive compression for reusability

2017-10-08 Thread Eli Schwartz
This allows for more easily extending the list of allowed compression methods, as it has to be modified in only one place. Also allow the user to specify their own preferred command + options for source packages in addition to compiled packages. Currently, makepkg.conf(5) erroneously claims this

[pacman-dev] [PATCH] makepkg: tell the compiler to record debugging info for debug packages

2017-10-08 Thread Eli Schwartz
In commit 8b0d59b83a60eb504567590346119fe4cd891cad support was added for storing the source files of binaries in debug packages. This made use of the debugedit program which is part of the RPM package manager, which is not very standalone. The same effect can be achieved using -fdebug-prefix-map,

Re: [pacman-dev] [PATCH] makepkg: refactor checking for write permissions into a utility function

2017-10-08 Thread Allan McRae
On 08/10/17 17:05, Eli Schwartz wrote: > Additionally provide a separate error for the confusing if unlikely > event that the user tries to use an existing file as a package output > directory. > > This also means we now consistently try to create any nonexistent *DEST > directories as needed