- Pacman-dev - lists.archlinux.org

[pacman-dev] [PATCH] pactest: set package tar format to GNU_FORMAT
by Allan McRae 12 Nov '19

12 Nov '19

python-3.8 changed the default tar format to PAX_FORMAT. This caused issues in our testsuite with package extraction of files with UTF-8 characters as we run the tests under the C locale. sycn600.py: error: error while reading package /tmp/pactest-xuhri4xa/var/cache/pacman/pkg/unicodechars-2.0-1.pkg.tar.gz: Pathname can't be converted from UTF-8 to current locale. Set format back to GNU_FORMAT. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- This patch at least gets us back to where we were with python-3.7. We probably have heaps more issues with UTF-8 usage... test/pacman/pmpkg.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/pacman/pmpkg.py b/test/pacman/pmpkg.py index 6a845222..e40868cc 100644 --- a/test/pacman/pmpkg.py +++ b/test/pacman/pmpkg.py @@ -142,7 +142,7 @@ def makepkg(self, path): util.mkdir(os.path.dirname(self.path)) # Generate package metadata - tar = tarfile.open(self.path, "w:gz") + tar = tarfile.open(self.path, "w:gz", format=tarfile.GNU_FORMAT) for name, data in archive_files: info = tarfile.TarInfo(name) info.size = len(data) -- 2.24.0

1 0

[pacman-dev] [PATCH] meson: make non-symlink scripts install for real, and use a better wrapper
by Eli Schwartz 12 Nov '19

12 Nov '19

We now generate the scripts using their real name, install them using meson's builtin facility instead of an install_script, and generate the wrapper scripts in the root of the build directory, instead of a subdirectory. This gets us closer to resolving FS#64394. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> --- This sort of seems to work, but passing variables around between different meson.build files feels awkward. On the plus side, fewer build-aux/ scripts! Is this a desirable experience? We now create all the wrapped scripts in the root of the build tree, which matches what we do for the pacman binaries and so on, and I guess this should be fine since unlike autotools we do not need to care about dirtying the source tree (we cannot do in-tree builds). It is more convenient accessing the scripts from there, too, and in fact we now create wrappers for repo-elephant (long may he live), pacman-key, pacman-db-upgrade, etc as well. build-aux/meson-install-script.sh | 6 ----- meson.build | 16 ++++++++++++- scripts/meson.build | 38 ++++++++++++++----------------- 3 files changed, 32 insertions(+), 28 deletions(-) delete mode 100644 build-aux/meson-install-script.sh diff --git a/build-aux/meson-install-script.sh b/build-aux/meson-install-script.sh deleted file mode 100644 index f5a42fca..00000000 --- a/build-aux/meson-install-script.sh +++ /dev/null @@ -1,6 +0,0 @@ -#!/bin/sh - -built_script=$1 -dest_path=$2 - -install -Dm755 "$built_script" "$DESTDIR/$dest_path" diff --git a/meson.build b/meson.build index 2c9185a6..572526b2 100644 --- a/meson.build +++ b/meson.build @@ -32,7 +32,6 @@ SED = find_program('sed') DU = find_program('du') LDCONFIG = get_option('ldconfig') MESON_MAKE_SYMLINK = join_paths(meson.source_root(), 'build-aux/meson-make-symlink.sh') -MESON_INSTALL_SCRIPT = join_paths(meson.source_root(), 'build-aux/meson-install-script.sh') BASH = find_program('bash4', 'bash') if BASH.found() @@ -366,6 +365,21 @@ executable( install : true, ) +foreach wrapper : script_wrappers + cdata = configuration_data() + cdata.set_quoted('BASH', BASH.path()) + cdata.set_quoted('BUILDDIR', wrapper[2]) + cdata.set_quoted('REAL_PROGPATH', wrapper[1].full_path()) + + # Create a wrapper script that bootstraps the real script within the build + # directory. Use configure_file instead of a custom_target to ensure that + # permissions on the input script wrapper are preserved. + configure_file( + input : join_paths(meson.source_root(), 'build-aux', 'script-wrapper.sh.in'), + output : wrapper[0], + configuration : cdata) +endforeach + configure_file( input : 'etc/makepkg.conf.in', output : 'makepkg.conf', diff --git a/scripts/meson.build b/scripts/meson.build index 696d8ddd..d2466523 100644 --- a/scripts/meson.build +++ b/scripts/meson.build @@ -26,42 +26,38 @@ foreach script : scripts install_dir : get_option('bindir')) endforeach +script_wrappers = [] foreach script : wrapped_scripts script_shortname = script.split('.')[0] - # Build the script, but don't install it. We want to keep it as a "private" - # artifact that we reference from a wrapper script in order to bootstrap it - # the build directory. internal_script = custom_target( script, input : script, command : [ SCRIPT_EDITOR, '@INPUT@', '@OUTPUT@', '0755'], - output : script, - build_by_default : true) - - cdata = configuration_data() - cdata.set_quoted('BASH', BASH.path()) - cdata.set_quoted('BUILDDIR', meson.current_build_dir()) - cdata.set_quoted('REAL_PROGPATH', internal_script.full_path()) - - # Create a wrapper script that bootstraps the real script within the build - # directory. Use configure_file instead of a custom_target to ensure that - # permissions on the input script wrapper are preserved. - configure_file( - input : join_paths(meson.source_root(), 'build-aux', 'script-wrapper.sh.in'), output : script_shortname, - configuration : cdata) + install : true, + install_dir : BINDIR) + + script_wrappers += [[ script_shortname, internal_script, meson.current_build_dir() ]] - # Install the real script - meson.add_install_script(MESON_INSTALL_SCRIPT, - internal_script.full_path(), - join_paths(BINDIR, script_shortname)) + if script_shortname == 'repo-add' + repo_add = internal_script + endif endforeach foreach symlink : ['repo-remove', 'repo-elephant'] meson.add_install_script(MESON_MAKE_SYMLINK, 'repo-add', join_paths(BINDIR, symlink)) + + internal_script = custom_target( + symlink, + build_by_default : true, + command : ['ln', '-sf', 'repo-add', '@OUTPUT@'], + depends : repo_add, + output : symlink) + + script_wrappers += [[ symlink, internal_script, meson.current_build_dir() ]] endforeach subdir('libmakepkg') -- 2.24.0

1 1

[pacman-dev] [PATCH] libalpm/sync.c: Do not download missing keys multiple times
by Allan McRae 11 Nov '19

11 Nov '19

We now store key structs of our missing key info, so can not search the list for string matches. This caused missing keys to be downloaded once for every package they signed. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- lib/libalpm/sync.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 59108eb9..70c37890 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -873,6 +873,14 @@ finish: } #ifdef HAVE_LIBGPGME + +static int key_cmp(const void *k1, const void *k2) { + const struct keyinfo_t *key1 = k1; + const char *key2 = k2; + + return strcmp(key1->keyid, key2); +} + static int check_keyring(alpm_handle_t *handle) { size_t current = 0, numtargs; @@ -910,7 +918,7 @@ static int check_keyring(alpm_handle_t *handle) alpm_list_t *k; for(k = keys; k; k = k->next) { char *key = k->data; - if(!alpm_list_find_str(errors, key) && + if(!alpm_list_find(errors, key, key_cmp) && _alpm_key_in_keychain(handle, key) == 0) { keyinfo = malloc(sizeof(struct keyinfo_t)); if(!keyinfo) { @@ -940,7 +948,7 @@ static int check_keyring(alpm_handle_t *handle) alpm_list_t *k; for(k = errors; k; k = k->next) { keyinfo = k->data; - if(_alpm_key_import(handle, keyinfo->uid, keyinfo->keyid) == -1) { + if(_alpm_key_import(handle, keyinfo->uid, keyinfo->keyid) == -1) { fail = 1; } free(keyinfo->uid); -- 2.24.0

1 0

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v5.2.1-13-g27f354a7
by Allan McRae 10 Nov '19

10 Nov '19

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 27f354a7874b965bf223832bdf9749504cd1a590 (commit) via 94982d0061890529cc22401b6763fb5e9ef4e7d1 (commit) via 1df22d3242a5db640a83124402249d34ac2e61e9 (commit) via 7b73c4a75fbddcfa1bc552fed49c228b9cdaf7b0 (commit) via 8c7043390f80d93445f2817ae5419b06fc72983a (commit) via 2dd7725f2a0cccd0a248531551c9fbea4ef35f1f (commit) from 3a814ee6bca9ee24a868c0dc032b321048a53e08 (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 27f354a7874b965bf223832bdf9749504cd1a590 Author: morganamilo <morganamilo(a)gmail.com> Date: Wed Nov 6 01:42:11 2019 +0000 pacman+libalpm: handle search errors Previously, pacman treated no matches and an error during search the same. To fix this, alpm_db_search now returns its status as an int and instead takes the to be returned list as a param. Allowing front ends to easily differentiate between errors and no matches. Signed-off-by: morganamilo <morganamilo(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 94982d0061890529cc22401b6763fb5e9ef4e7d1 Author: morganamilo <morganamilo(a)gmail.com> Date: Wed Nov 6 00:37:26 2019 +0000 pacman: make exact_file an int We only ever use it as a bool, no need to pass a char* around. Signed-off-by: morganamilo <morganamilo(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 1df22d3242a5db640a83124402249d34ac2e61e9 Author: Ethan Sommer <e5ten.arch(a)gmail.com> Date: Wed Nov 6 20:23:08 2019 -0500 repo-add: use wc -c on stdin instead of file to avoid use of cut Redirect file to stdin so wc -c doesn't print a file name that needs to be stripped. Signed-off-by: Ethan Sommer <e5ten.arch(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 7b73c4a75fbddcfa1bc552fed49c228b9cdaf7b0 Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Wed Nov 6 00:26:38 2019 -0500 build: remove use of handcrafted m4 in configuring scripts Now that library/ is fully gone, we don't need this anymore. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 8c7043390f80d93445f2817ae5419b06fc72983a Author: Eli Schwartz <eschwartz(a)archlinux.org> Date: Wed Nov 6 00:05:22 2019 -0500 scripts/library: remove human_to_size pkgdelta was the last user, and it is gone now. Signed-off-by: Eli Schwartz <eschwartz(a)archlinux.org> Signed-off-by: Allan McRae <allan(a)archlinux.org> commit 2dd7725f2a0cccd0a248531551c9fbea4ef35f1f Author: Allan McRae <allan(a)archlinux.org> Date: Wed Nov 6 15:46:59 2019 +1000 Remove "Generated from ...; do not edit by hand" from scripts This is a useless piece of information. Signed-off-by: Allan McRae <allan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: Makefile.am | 1 - build-aux/edit-script.sh.in | 1 - lib/libalpm/alpm.h | 6 +++-- lib/libalpm/db.c | 24 +++++++++-------- lib/libalpm/db.h | 3 ++- meson.build | 2 -- scripts/Makefile.am | 9 ++----- scripts/library/README | 10 ------- scripts/library/human_to_size.sh | 51 ----------------------------------- scripts/makepkg-template.pl.in | 1 - scripts/makepkg.sh.in | 1 - scripts/meson.build | 16 ++--------- scripts/pacman-db-upgrade.sh.in | 1 - scripts/pacman-key.sh.in | 1 - scripts/po/POTFILES.in | 1 - scripts/repo-add.sh.in | 5 ++-- src/pacman/files.c | 10 +++---- src/pacman/package.c | 7 +++-- src/pacman/query.c | 10 ++++++- src/pacman/sync.c | 10 ++++++- test/scripts/Makefile.am | 3 +-- test/scripts/human_to_size_test.sh | 55 -------------------------------------- test/scripts/meson.build | 1 - 23 files changed, 54 insertions(+), 175 deletions(-) delete mode 100644 scripts/library/README delete mode 100644 scripts/library/human_to_size.sh delete mode 100755 test/scripts/human_to_size_test.sh hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] pacman: print error message for -F with invalid regex
by morganamilo 09 Nov '19

09 Nov '19

diff --git a/src/pacman/files.c b/src/pacman/files.c index 19191dd8..05b6f223 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -115,7 +115,8 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { if(regex) { if(regcomp(&reg, targ, REG_EXTENDED | REG_NOSUB | REG_ICASE | REG_NEWLINE) != 0) { - /* TODO: error message */ + pm_printf(ALPM_LOG_ERROR, + _("search failed: invalid regular expression '%s'\n"), targ); goto notfound; } } -- 2.23.0

3 2

[pacman-dev] [PATCH] Warn about skipping invalid regex with -Fx
by Allan McRae 08 Nov '19

08 Nov '19

Print a warning when -Fx encounters a broken regex. These were previosly skipped silently as when providing -Fx with multiple arguments, a file that matches any of the arguments is returned. This means encountering one broken argument is not an error. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- src/pacman/files.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/pacman/files.c b/src/pacman/files.c index 3b6dc23b..fa3d9651 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -115,7 +115,7 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { if(regex) { if(regcomp(&reg, targ, REG_EXTENDED | REG_NOSUB | REG_ICASE | REG_NEWLINE) != 0) { - /* TODO: error message */ + pm_printf(ALPM_LOG_WARNING, "skipping invalid regex: '%s'\n", targ); goto notfound; } } -- 2.23.0

1 0

[pacman-dev] [PATCH] pacman+libalpm: handle search errors
by morganamilo 08 Nov '19

08 Nov '19

Previously, pacman treated no matches and an error during search the same. To fix this, alpm_db_search now returns its status as an int and instead takes the to be returned list as a param. Allowing front ends to easily differentiate between errors and no matches. Signed-off-by: morganamilo <morganamilo(a)gmail.com> diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 4486da44..d3630385 100644 --- a/lib/libalpm/alpm.h +++ b/lib/libalpm/alpm.h @@ -997,9 +997,11 @@ alpm_list_t *alpm_db_get_groupcache(alpm_db_t *db); /** Searches a database with regular expressions. * @param db pointer to the package database to search in * @param needles a list of regular expressions to search for - * @return the list of packages matching all regular expressions on success, NULL on error + * @param ret the list of packages matching all regular expressions + * @return 0 on success, -1 on error (pm_errno is set accordingly) */ -alpm_list_t *alpm_db_search(alpm_db_t *db, const alpm_list_t *needles); +int alpm_db_search(alpm_db_t *db, const alpm_list_t *needles, + alpm_list_t **ret); typedef enum _alpm_db_usage_t { ALPM_DB_USAGE_SYNC = 1, diff --git a/lib/libalpm/db.c b/lib/libalpm/db.c index a443e552..cf4c865f 100644 --- a/lib/libalpm/db.c +++ b/lib/libalpm/db.c @@ -298,12 +298,14 @@ alpm_list_t SYMEXPORT *alpm_db_get_groupcache(alpm_db_t *db) } /** Searches a database. */ -alpm_list_t SYMEXPORT *alpm_db_search(alpm_db_t *db, const alpm_list_t *needles) +int SYMEXPORT alpm_db_search(alpm_db_t *db, const alpm_list_t *needles, + alpm_list_t **ret) { - ASSERT(db != NULL, return NULL); + ASSERT(db != NULL && ret != NULL && *ret == NULL, + RET_ERR(db->handle, ALPM_ERR_WRONG_ARGS, -1)); db->handle->pm_errno = ALPM_ERR_OK; - return _alpm_db_search(db, needles); + return _alpm_db_search(db, needles, ret); } /** Sets the usage bitmask for a repo */ @@ -396,13 +398,13 @@ int _alpm_db_cmp(const void *d1, const void *d2) return strcmp(db1->treename, db2->treename); } -alpm_list_t *_alpm_db_search(alpm_db_t *db, const alpm_list_t *needles) +int _alpm_db_search(alpm_db_t *db, const alpm_list_t *needles, + alpm_list_t **ret) { const alpm_list_t *i, *j, *k; - alpm_list_t *ret = NULL; if(!(db->usage & ALPM_DB_USAGE_SEARCH)) { - return NULL; + return 0; } /* copy the pkgcache- we will free the list var after each needle */ @@ -415,12 +417,12 @@ alpm_list_t *_alpm_db_search(alpm_db_t *db, const alpm_list_t *needles) if(i->data == NULL) { continue; } - ret = NULL; + *ret = NULL; targ = i->data; _alpm_log(db->handle, ALPM_LOG_DEBUG, "searching for target '%s'\n", targ); if(regcomp(&reg, targ, REG_EXTENDED | REG_NOSUB | REG_ICASE | REG_NEWLINE) != 0) { - RET_ERR(db->handle, ALPM_ERR_INVALID_REGEX, NULL); + RET_ERR(db->handle, ALPM_ERR_INVALID_REGEX, -1); } for(j = list; j; j = j->next) { @@ -463,18 +465,18 @@ alpm_list_t *_alpm_db_search(alpm_db_t *db, const alpm_list_t *needles) _alpm_log(db->handle, ALPM_LOG_DEBUG, "search target '%s' matched '%s' on package '%s'\n", targ, matched, name); - ret = alpm_list_add(ret, pkg); + *ret = alpm_list_add(*ret, pkg); } } /* Free the existing search list, and use the returned list for the * next needle. This allows for AND-based package searching. */ alpm_list_free(list); - list = ret; + list = *ret; regfree(&reg); } - return ret; + return 0; } /* Returns a new package cache from db. diff --git a/lib/libalpm/db.h b/lib/libalpm/db.h index f17444e6..e7ad98f5 100644 --- a/lib/libalpm/db.h +++ b/lib/libalpm/db.h @@ -87,7 +87,8 @@ alpm_db_t *_alpm_db_new(const char *treename, int is_local); void _alpm_db_free(alpm_db_t *db); const char *_alpm_db_path(alpm_db_t *db); int _alpm_db_cmp(const void *d1, const void *d2); -alpm_list_t *_alpm_db_search(alpm_db_t *db, const alpm_list_t *needles); +int _alpm_db_search(alpm_db_t *db, const alpm_list_t *needles, + alpm_list_t **ret); alpm_db_t *_alpm_db_register_local(alpm_handle_t *handle); alpm_db_t *_alpm_db_register_sync(alpm_handle_t *handle, const char *treename, int level); diff --git a/src/pacman/package.c b/src/pacman/package.c index 35cfcd94..ec6e78fc 100644 --- a/src/pacman/package.c +++ b/src/pacman/package.c @@ -518,12 +518,13 @@ void print_groups(alpm_pkg_t *pkg) * @param db the database we're searching * @param targets the targets we're searching for * @param show_status show if the package is also in the local db + * @return -1 on error, 0 if there were matches, 1 if there were not */ int dump_pkg_search(alpm_db_t *db, alpm_list_t *targets, int show_status) { int freelist = 0; alpm_db_t *db_local; - alpm_list_t *i, *searchlist; + alpm_list_t *i, *searchlist = NULL; unsigned short cols; const colstr_t *colstr = &config->colstr; @@ -533,7 +534,9 @@ int dump_pkg_search(alpm_db_t *db, alpm_list_t *targets, int show_status) /* if we have a targets list, search for packages matching it */ if(targets) { - searchlist = alpm_db_search(db, targets); + if(alpm_db_search(db, targets, &searchlist) != 0) { + return -1; + } freelist = 1; } else { searchlist = alpm_db_get_pkgcache(db); diff --git a/src/pacman/query.c b/src/pacman/query.c index 2b9ee7a6..bdcfc3c1 100644 --- a/src/pacman/query.c +++ b/src/pacman/query.c @@ -233,7 +233,15 @@ targcleanup: static int query_search(alpm_list_t *targets) { alpm_db_t *db_local = alpm_get_localdb(config->handle); - return dump_pkg_search(db_local, targets, 0); + int ret = dump_pkg_search(db_local, targets, 0); + if(ret == -1) { + alpm_errno_t err = alpm_errno(config->handle); + pm_printf(ALPM_LOG_ERROR, "search failed: %s\n", alpm_strerror(err)); + return 1; + } + + return ret; + } static unsigned short pkg_get_locality(alpm_pkg_t *pkg) diff --git a/src/pacman/sync.c b/src/pacman/sync.c index 9033cee7..4bdeff7b 100644 --- a/src/pacman/sync.c +++ b/src/pacman/sync.c @@ -311,7 +311,15 @@ static int sync_search(alpm_list_t *syncs, alpm_list_t *targets) for(i = syncs; i; i = alpm_list_next(i)) { alpm_db_t *db = i->data; - found += !dump_pkg_search(db, targets, 1); + int ret = dump_pkg_search(db, targets, 1); + + if(ret == -1) { + alpm_errno_t err = alpm_errno(config->handle); + pm_printf(ALPM_LOG_ERROR, "search failed: %s\n", alpm_strerror(err)); + return 1; + } + + found += !ret; } return (found == 0); -- 2.23.0

3 2

[pacman-dev] [PATCH] repo-add: use wc -c on stdin instead of file to avoid use of cut
by Ethan Sommer 07 Nov '19

07 Nov '19

Redirect file to stdin so wc -c doesn't print a file name that needs to be stripped. Signed-off-by: Ethan Sommer <e5ten.arch(a)gmail.com> --- scripts/repo-add.sh.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in index caf1232d..7b2a9edb 100644 --- a/scripts/repo-add.sh.in +++ b/scripts/repo-add.sh.in @@ -273,7 +273,7 @@ db_write_entry() { error "$(gettext "Cannot use armored signatures for packages: %s")" "$pkgfile.sig" return 1 fi - pgpsigsize=$(wc -c "$pkgfile.sig" | cut -d' ' -f1) + pgpsigsize=$(wc -c < "$pkgfile.sig") if (( pgpsigsize > 16384 )); then error "$(gettext "Invalid package signature file '%s'.")" "$pkgfile.sig" return 1 @@ -282,7 +282,7 @@ db_write_entry() { pgpsig=$(base64 "$pkgfile.sig" | tr -d '\n') fi - csize=$(wc -c "$pkgfile" | cut -d' ' -f1) + csize=$(wc -c < "$pkgfile") # compute checksums msg2 "$(gettext "Computing checksums...")" -- 2.23.0

2 1

[pacman-dev] [PATCH] pacman: make exact_file an int
by morganamilo 07 Nov '19

07 Nov '19

We only ever use it as a bool, no need to pass a char* around. Signed-off-by: morganamilo <morganamilo(a)gmail.com> diff --git a/src/pacman/files.c b/src/pacman/files.c index 19191dd8..3b6dc23b 100644 --- a/src/pacman/files.c +++ b/src/pacman/files.c @@ -58,7 +58,7 @@ static void print_owned_by(alpm_db_t *db, alpm_pkg_t *pkg, char *filename) alpm_pkg_get_version(pkg), colstr->nocolor); } -static void print_match(alpm_list_t *match, alpm_db_t *repo, alpm_pkg_t *pkg, char *exact_file) +static void print_match(alpm_list_t *match, alpm_db_t *repo, alpm_pkg_t *pkg, int exact_file) { alpm_db_t *db_local = alpm_get_localdb(config->handle); const colstr_t *colstr = &config->colstr; @@ -71,7 +71,7 @@ static void print_match(alpm_list_t *match, alpm_db_t *repo, alpm_pkg_t *pkg, ch } } else if(config->quiet) { printf("%s/%s\n", alpm_db_get_name(repo), alpm_pkg_get_name(pkg)); - } else if(exact_file != NULL) { + } else if(exact_file) { alpm_list_t *ml; for(ml = match; ml; ml = alpm_list_next(ml)) { char *filename = ml->data; @@ -104,9 +104,9 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { int found = 0; regex_t reg; size_t len = strlen(targ); - char *exact_file = strchr(targ, '/'); + int exact_file = strchr(targ, '/') != NULL; - if(exact_file != NULL) { + if(exact_file) { while(len > 1 && targ[0] == '/') { targ++; len--; @@ -131,7 +131,7 @@ static int files_search(alpm_list_t *syncs, alpm_list_t *targets, int regex) { alpm_filelist_t *files = alpm_pkg_get_files(pkg); alpm_list_t *match = NULL; - if(exact_file != NULL) { + if(exact_file) { if (regex) { for(size_t f = 0; f < files->count; f++) { char *c = files->files[f].name; -- 2.23.0

2 1

[pacman-dev] [PATCH] Remove "Generated from ...; do not edit by hand" from scripts
by Allan McRae 06 Nov '19

06 Nov '19

This is a useless piece of information. Signed-off-by: Allan McRae <allan(a)archlinux.org> --- build-aux/edit-script.sh.in | 1 - scripts/Makefile.am | 3 +-- scripts/makepkg-template.pl.in | 1 - scripts/makepkg.sh.in | 1 - scripts/pacman-db-upgrade.sh.in | 1 - scripts/pacman-key.sh.in | 1 - scripts/repo-add.sh.in | 1 - 7 files changed, 1 insertion(+), 8 deletions(-) diff --git a/build-aux/edit-script.sh.in b/build-aux/edit-script.sh.in index 7423a223..661c22d5 100644 --- a/build-aux/edit-script.sh.in +++ b/build-aux/edit-script.sh.in @@ -20,7 +20,6 @@ mode=$3 -e "s|@DEBUGSUFFIX[@]|@DEBUGSUFFIX@|g" \ -e "s|@INODECMD[@]|@INODECMD@|g" \ -e "s|@FILECMD[@]|@FILECMD@|g" \ - -e "s|@configure_input[@]|Generated from ${input##*/}; do not edit by hand.|g" \ "$input" >"$output" if [[ $mode ]]; then diff --git a/scripts/Makefile.am b/scripts/Makefile.am index 3cb6c133..191fc3d3 100644 --- a/scripts/Makefile.am +++ b/scripts/Makefile.am @@ -186,8 +186,7 @@ edit = sed \ -e 's|@DEBUGSUFFIX[@]|$(DEBUGSUFFIX)|g' \ -e "s|@INODECMD[@]|$(INODECMD)|g" \ -e "s|@FILECMD[@]|$(FILECMD)|g" \ - -e 's|@SCRIPTNAME[@]|$@|g' \ - -e 's|@configure_input[@]|Generated from $<; do not edit by hand.|g' + -e 's|@SCRIPTNAME[@]|$@|g' ## All the scripts depend on Makefile so that they are rebuilt when the ## prefix etc. changes. Use chmod -w to prevent people from editing the diff --git a/scripts/makepkg-template.pl.in b/scripts/makepkg-template.pl.in index da5eba1b..762dd4be 100755 --- a/scripts/makepkg-template.pl.in +++ b/scripts/makepkg-template.pl.in @@ -1,6 +1,5 @@ #!/usr/bin/perl # makepkg-template - template system for makepkg -# @configure_input@ # # Copyright (c) 2013-2019 Pacman Development Team <pacman-dev(a)archlinux.org> # diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index 22df1b9f..2deb61da 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1,7 +1,6 @@ #!/bin/bash # # makepkg - make packages compatible for use with pacman -# @configure_input@ # # Copyright (c) 2006-2019 Pacman Development Team <pacman-dev(a)archlinux.org> # Copyright (c) 2002-2006 by Judd Vinet <jvinet(a)zeroflux.org> diff --git a/scripts/pacman-db-upgrade.sh.in b/scripts/pacman-db-upgrade.sh.in index 5d65bc88..4a904b6d 100644 --- a/scripts/pacman-db-upgrade.sh.in +++ b/scripts/pacman-db-upgrade.sh.in @@ -1,7 +1,6 @@ #!/bin/bash -e # # pacman-db-upgrade - upgrade the local pacman db to a newer format -# @configure_input@ # # Copyright (c) 2010-2019 Pacman Development Team <pacman-dev(a)archlinux.org> # diff --git a/scripts/pacman-key.sh.in b/scripts/pacman-key.sh.in index 366fd205..613b3514 100644 --- a/scripts/pacman-key.sh.in +++ b/scripts/pacman-key.sh.in @@ -2,7 +2,6 @@ # # pacman-key - manages pacman's keyring # Based on apt-key, from Debian -# @configure_input@ # # Copyright (c) 2010-2019 Pacman Development Team <pacman-dev(a)archlinux.org> # diff --git a/scripts/repo-add.sh.in b/scripts/repo-add.sh.in index caf1232d..f7dd8948 100644 --- a/scripts/repo-add.sh.in +++ b/scripts/repo-add.sh.in @@ -2,7 +2,6 @@ # # repo-add - add a package to a given repo database file # repo-remove - remove a package entry from a given repo database file -# @configure_input@ # # Copyright (c) 2006-2019 Pacman Development Team <pacman-dev(a)archlinux.org> # -- 2.23.0

1 0