May 2021 - Pacman-dev - lists.archlinux.org

[pacman-dev] Translation of man pages
by Mario Blättermann 19 Sep '21

19 Sep '21

Hello, the translation help page [1] says: --- There are currently no efforts underway to include translated manual pages in the pacman codebase. However, this is not to say translations are unwelcome. If someone has experience with i18n man pages and how to best include them with our source, please contact the pacman-dev mailing list at pacman-dev(a)archlinux.org. --- OK, then let me introduce my approach. Although the man page translations are already maintained in the manpages-l10n project [2] for some years, I think it is it better to use Pacman's upstream tree instead. The external maintenance has some disadvantages: We download distribution packages, create the templates, translate them and release the translated man pages some months later. Besides this delay, we don't know how many users know about the existence of the translated versions. Having the translations in the upstream project (translated using Transifex) would be better, and the localized versions would be released with Pacman itself and always in sync with the English ones. See the attached patch and config file. The patch manpage-markup.diff fixes the Asciidoc markup to get Po4a [3] properly working. After applying the patch, put po4a.cfg in doc/po/. I've imported German, Brazilian Portuguese and French translations from manpages-l10n, but even compressed they are too big for the list. The .po files will follow in separate mails.To test the creation of the translated Asciidoc files, just run 'po4a po4a.cfg'. This command creates a translation template and a subdirectory for each language and saves the resulting files there. Once you have the Asciidoc files, they can be processed further like those in doc/. Unfortunately, I don't have neither Autotools nor Meson knowledge, maybe someone else can help with integration into the build systems? [1] https://archlinux.org/pacman/translation-help.html#_translating_manpages [2] https://salsa.debian.org/manpages-l10n-team/manpages-l10n [3] https://po4a.org/index.php.en Best Regards, Mario

2 5

[pacman-dev] [PATCH 1/2] libalpm: take alpm_trans_t out of the public API
by morganamilo 03 Jun '21

03 Jun '21

this type is only used internally by alpm --- lib/libalpm/alpm.h | 6 ------ lib/libalpm/handle.h | 1 + lib/libalpm/trans.h | 4 ++-- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/lib/libalpm/alpm.h b/lib/libalpm/alpm.h index 101d686b..1413263b 100644 --- a/lib/libalpm/alpm.h +++ b/lib/libalpm/alpm.h @@ -113,12 +113,6 @@ typedef struct __alpm_db_t alpm_db_t; */ typedef struct __alpm_pkg_t alpm_pkg_t; -/** Transaction structure used internally by libalpm - * @ingroup libalpm_trans - * */ -typedef struct __alpm_trans_t alpm_trans_t; - - /** The time type used by libalpm. Represents a unix time stamp * @ingroup libalpm_misc */ typedef int64_t alpm_time_t; diff --git a/lib/libalpm/handle.h b/lib/libalpm/handle.h index b1526c67..f2602057 100644 --- a/lib/libalpm/handle.h +++ b/lib/libalpm/handle.h @@ -26,6 +26,7 @@ #include "alpm_list.h" #include "alpm.h" +#include "trans.h" #ifdef HAVE_LIBCURL #include <curl/curl.h> diff --git a/lib/libalpm/trans.h b/lib/libalpm/trans.h index 9e7fcbd7..d2ec0223 100644 --- a/lib/libalpm/trans.h +++ b/lib/libalpm/trans.h @@ -36,7 +36,7 @@ typedef enum _alpm_transstate_t { } alpm_transstate_t; /* Transaction */ -struct __alpm_trans_t { +typedef struct _alpm_trans_t { /* bitfield of alpm_transflag_t flags */ int flags; alpm_transstate_t state; @@ -44,7 +44,7 @@ struct __alpm_trans_t { alpm_list_t *add; /* list of (alpm_pkg_t *) */ alpm_list_t *remove; /* list of (alpm_pkg_t *) */ alpm_list_t *skip_remove; /* list of (char *) */ -}; +} alpm_trans_t; void _alpm_trans_free(alpm_trans_t *trans); /* flags is a bitfield of alpm_transflag_t flags */ -- 2.31.1

2 4

[pacman-dev] [PATCH v2] docs: Update submission to lists.archlinux.org
by Morten Linderud 03 Jun '21

03 Jun '21

From: Morten Linderud <morten(a)linderud.pw> Currently some overly aggressive spam filter is going to complain if there is a mismatch between the To and Reply-To header. Since Arch Linux has changed the mailing list domains this changes the documentation to reflect the change. https://archlinux.org/news/arch-linux-mailing-list-id-changes/ Signed-off-by: Morten Linderud <morten(a)linderud.pw> --- doc/index.asciidoc | 4 ++-- doc/translation-help.asciidoc | 4 ++-- lib/libalpm/po/meson.build | 2 +- scripts/po/meson.build | 2 +- src/pacman/po/meson.build | 2 +- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/doc/index.asciidoc b/doc/index.asciidoc index 87f07dda..7200bc18 100644 --- a/doc/index.asciidoc +++ b/doc/index.asciidoc @@ -252,7 +252,7 @@ Utilities available: Bugs ---- If you find bugs (which is quite likely), please email them to the pacman-dev -mailing last at mailto:pacman-dev@archlinux.org[] with specific information +mailing last at mailto:pacman-dev@lists.archlinux.org[] with specific information such as your command-line, the nature of the bug, and even the package database if it helps. @@ -263,6 +263,6 @@ bugs under the Pacman project. Copyright --------- pacman is Copyright (C) 2006-2021 Pacman Development Team -<pacman-dev(a)archlinux.org> and Copyright (C) 2002-2006 Judd Vinet +<pacman-dev(a)lists.archlinux.org> and Copyright (C) 2002-2006 Judd Vinet <jvinet(a)zeroflux.org> and is licensed through the GNU General Public License, version 2 or later. diff --git a/doc/translation-help.asciidoc b/doc/translation-help.asciidoc index 5a67a2d6..fe020e92 100644 --- a/doc/translation-help.asciidoc +++ b/doc/translation-help.asciidoc @@ -55,7 +55,7 @@ Pre-release Updates ~~~~~~~~~~~~~~~~~~~ A week or two before each release, the codebase will go into a string freeze -and an email will be sent to the mailto:pacman-dev@archlinux.org[pacman-dev] +and an email will be sent to the mailto:pacman-dev@lists.archlinux.org[pacman-dev] mailing list asking for translations. This email will have a prefix of *[translation]* for anyone looking to set up an email filter. @@ -150,4 +150,4 @@ There are currently no efforts underway to include translated manual pages in the pacman codebase. However, this is not to say translations are unwelcome. If someone has experience with i18n man pages and how to best include them with our source, please contact the pacman-dev mailing list at -mailto:pacman-dev@archlinux.org[]. +mailto:pacman-dev@lists.archlinux.org[]. diff --git a/lib/libalpm/po/meson.build b/lib/libalpm/po/meson.build index cec28a15..61db9ed9 100644 --- a/lib/libalpm/po/meson.build +++ b/lib/libalpm/po/meson.build @@ -3,7 +3,7 @@ i18n.gettext( args : [ '--directory=@0@'.format(meson.current_source_dir()), '--msgid-bugs-address=http://bugs.archlinux.org/index.php?project=3', - '--copyright-holder="Pacman Development Team <pacman-dev(a)archlinux.org>"', + '--copyright-holder="Pacman Development Team <pacman-dev(a)lists.archlinux.org>"', '--language', 'c', '--keyword=_', diff --git a/scripts/po/meson.build b/scripts/po/meson.build index d8b8c51c..874b51dc 100644 --- a/scripts/po/meson.build +++ b/scripts/po/meson.build @@ -3,7 +3,7 @@ i18n.gettext( args : [ '--directory=@0@'.format(meson.current_source_dir()), '--msgid-bugs-address=http://bugs.archlinux.org/index.php?project=3', - '--copyright-holder="Pacman Development Team <pacman-dev(a)archlinux.org>"', + '--copyright-holder="Pacman Development Team <pacman-dev(a)lists.archlinux.org>"', '--language', 'shell', '--keyword=_', diff --git a/src/pacman/po/meson.build b/src/pacman/po/meson.build index eb45fa1c..78c0230f 100644 --- a/src/pacman/po/meson.build +++ b/src/pacman/po/meson.build @@ -3,7 +3,7 @@ i18n.gettext( args : [ '--directory=@0@'.format(meson.current_source_dir()), '--msgid-bugs-address=http://bugs.archlinux.org/index.php?project=3', - '--copyright-holder="Pacman Development Team <pacman-dev(a)archlinux.org>"', + '--copyright-holder="Pacman Development Team <pacman-dev(a)lists.archlinux.org>"', '--language', 'c', '--keyword=_', -- 2.31.1

3 2

[pacman-dev] Questions about stderr/stdout behaviour on failures
by Oskar Roesler 02 Jun '21

02 Jun '21

Hello, when file conflicts in packages occur, why does the actual conflict get written to stdout and only an non-informative error message to stderr? Example: Stderr only: warning: dependency cycle detected: warning: libglvnd will be installed before its mesa dependency error: failed to commit transaction (conflicting files) ==> ERROR: 'pacman' failed to install missing dependencies. warning: dependency cycle detected: warning: harfbuzz will be installed before its freetype2 dependency warning: dependency cycle detected: warning: libglvnd will be installed before its mesa dependency error: failed to commit transaction (conflicting files) ==> ERROR: 'pacman' failed to install missing dependencies. Both outputs combined: :: Proceed with installation? [Y/n] checking keyring... checking package integrity... loading package files... checking for file conflicts... error: failed to commit transaction (conflicting files) /opt/ros/melodic/lib/python3.9/site-packages/libgtest.so exists in both 'ros-melodic-moveit-core' and 'ros-melodic-tf' /opt/ros/melodic/lib/python3.9/site-packages/libgtest.so.1.10.0 exists in both 'ros-melodic-moveit-core' and 'ros-melodic-tf' Errors occurred, no packages were upgraded. ==> ERROR: 'pacman' failed to install missing dependencies. ==> Missing dependencies: -> ros-build-tools -> ros-melodic-message-filters -> ros-melodic-moveit-core -> ros-melodic-moveit-msgs As e.g. the dependency cycle errors get fully written to stderr, is this a bug or itended? Imho this one line more is important to stderr, otherwise the whole stderr log helps me as much as the returncode. Regards, Oskar

2 2

[pacman-dev] [PATCH 1/1] RFC: handle soft failure in downloads
by Christian Hesse 02 Jun '21

02 Jun '21

From: Christian Hesse <mail(a)eworm.de> A server can indicate a soft failure by returning 412 (Precondition Failed). Only a warning (instead of error) message is logged and download is retried with next server, without counting server error. This http code can be used by servers that are not expected to be complete, for example when serving a local cache. Signed-off-by: Christian Hesse <mail(a)eworm.de> --- lib/libalpm/dload.c | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index f6a4f012..7116cb03 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -515,13 +515,23 @@ static int curl_check_finished_download(CURLM *curlm, CURLMsg *msg, if(!payload->errors_ok) { handle->pm_errno = ALPM_ERR_RETRIEVE; - /* non-translated message is same as libcurl */ - snprintf(payload->error_buffer, sizeof(payload->error_buffer), + if(payload->respcode == 412) { + /* A server can indicate a soft failure by returning 412 + (Precondition Failed). Only a debug message is logged and + download is retried with next server, without counting + server error. */ + _alpm_log(handle, ALPM_LOG_WARNING, + _("file '%s' is currently not available from %s\n"), + payload->remote_name, hostname); + } else { + /* non-translated message is same as libcurl */ + snprintf(payload->error_buffer, sizeof(payload->error_buffer), "The requested URL returned error: %ld", payload->respcode); - _alpm_log(handle, ALPM_LOG_ERROR, + _alpm_log(handle, ALPM_LOG_ERROR, _("failed retrieving file '%s' from %s : %s\n"), payload->remote_name, hostname, payload->error_buffer); - server_soft_error(handle, payload->fileurl); + server_soft_error(handle, payload->fileurl); + } } if(curl_retry_next_server(curlm, curl, payload) == 0) { (*active_downloads_num)++;

6 19

[pacman-dev] [PATCH v2] libalpm: fix 404 pages ending up inside package downloads
by morganamilo 21 May '21

21 May '21

Previously successfull but non 200 responses would still be written to disk before trying the next server. This would lead to the next download resuming with the html response in the file. --- v2: ensure allow_resume is passed to callback Signed-off-by: morganamilo <morganamilo(a)archlinux.org> --- lib/libalpm/dload.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 2d8b4d6d..048cec5e 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -404,7 +404,7 @@ static FILE *create_tempfile(struct dload_payload *payload, const char *localpat } /* Return 0 if retry was successful, -1 otherwise */ -static int curl_retry_next_server(CURLM *curlm, CURL *curl, struct dload_payload *payload) +static int curl_retry_next_server(CURLM *curlm, CURL *curl, struct dload_payload *payload, int allow_resume) { const char *server; size_t len; @@ -428,10 +428,9 @@ static int curl_retry_next_server(CURLM *curlm, CURL *curl, struct dload_payload MALLOC(payload->fileurl, len, RET_ERR(handle, ALPM_ERR_MEMORY, -1)); snprintf(payload->fileurl, len, "%s/%s", server, payload->filepath); - fflush(payload->localf); - if(payload->allow_resume && stat(payload->tempfile_name, &st) == 0) { + if(allow_resume && stat(payload->tempfile_name, &st) == 0) { /* a previous partial download exists, resume from end of file. */ payload->tempfile_openmode = "ab"; curl_easy_setopt(curl, CURLOPT_RESUME_FROM_LARGE, (curl_off_t)st.st_size); @@ -449,7 +448,7 @@ static int curl_retry_next_server(CURLM *curlm, CURL *curl, struct dload_payload if(handle->dlcb && !payload->signature) { alpm_download_event_retry_t cb_data; - cb_data.resume = payload->allow_resume; + cb_data.resume = allow_resume; handle->dlcb(handle->dlcb_ctx, payload->remote_name, ALPM_DOWNLOAD_RETRY, &cb_data); } @@ -510,7 +509,7 @@ static int curl_check_finished_download(CURLM *curlm, CURLMsg *msg, payload->remote_name, hostname, payload->error_buffer); server_soft_error(handle, payload->fileurl); } - if(curl_retry_next_server(curlm, curl, payload) == 0) { + if(curl_retry_next_server(curlm, curl, payload, false) == 0) { (*active_downloads_num)++; return 2; } else { @@ -537,7 +536,7 @@ static int curl_check_finished_download(CURLM *curlm, CURLMsg *msg, _("failed retrieving file '%s' from %s : %s\n"), payload->remote_name, hostname, payload->error_buffer); server_hard_error(handle, payload->fileurl); - if(curl_retry_next_server(curlm, curl, payload) == 0) { + if(curl_retry_next_server(curlm, curl, payload, payload->allow_resume) == 0) { (*active_downloads_num)++; return 2; } else { @@ -556,7 +555,7 @@ static int curl_check_finished_download(CURLM *curlm, CURLMsg *msg, "failed retrieving file '%s' from %s : %s\n", payload->remote_name, hostname, payload->error_buffer); } - if(curl_retry_next_server(curlm, curl, payload) == 0) { + if(curl_retry_next_server(curlm, curl, payload, payload->allow_resume) == 0) { (*active_downloads_num)++; return 2; } else { -- 2.31.1

1 1

Re: [pacman-dev] Translation of man pages
by Mario Blättermann 21 May '21

21 May '21

Hello, see pt_BR.po attached. Best Regards, Mario

1 0

Re: [pacman-dev] Translation of man pages
by Mario Blättermann 21 May '21

21 May '21

Hello, see fr.po attached. Best Regards, Mario

1 0

[pacman-dev] [PATCH] libalpm: call retry events for sig downlods
by morganamilo 21 May '21

21 May '21

Around the same time retry events were added, there was a patch to pass sig download events to the frontend. The retry code was not updated to account for this. Signed-off-by: morganamilo <morganamilo(a)archlinux.org> diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index 048cec5e..37e8ae34 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -446,7 +446,7 @@ static int curl_retry_next_server(CURLM *curlm, CURL *curl, struct dload_payload fseek(payload->localf, 0, SEEK_SET); } - if(handle->dlcb && !payload->signature) { + if(handle->dlcb) { alpm_download_event_retry_t cb_data; cb_data.resume = allow_resume; handle->dlcb(handle->dlcb_ctx, payload->remote_name, ALPM_DOWNLOAD_RETRY, &cb_data); -- 2.31.1

1 0

[pacman-dev] [PATCH 1/1] doc/pacman.conf: ParallelDownloads expects a value
by Christian Hesse 20 May '21

20 May '21

From: Christian Hesse <mail(a)eworm.de> This can not be specified on its own but requires a value. Signed-off-by: Christian Hesse <mail(a)eworm.de> --- doc/pacman.conf.5.asciidoc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/pacman.conf.5.asciidoc b/doc/pacman.conf.5.asciidoc index fd765f3d..77a3907f 100644 --- a/doc/pacman.conf.5.asciidoc +++ b/doc/pacman.conf.5.asciidoc @@ -202,7 +202,7 @@ Options Disable defaults for low speed limit and timeout on downloads. Use this if you have issues downloading files with proxy and/or security gateway. -*ParallelDownloads*:: +*ParallelDownloads =* ...:: Specifies number of concurrent download streams. The value needs to be a positive integer. If this config option is not set then only one download stream is used (i.e. downloads happen sequentially).

2 1