- Pacman-dev - lists.archlinux.org

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v4.0.1-50-g9d1e808
by dan＠archlinux.org 31 Jan '12

31 Jan '12

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, maint has been updated via 9d1e8084dfa0a1ebae6c37e3b839289cb905116f (commit) via 7b2f600d717ce5ecd0025d2fba43d405adb58bd4 (commit) via 5c2928aa259cac1dcdaa8b27172c2ee12327d21a (commit) via 9e9835f4640a45cc758fa64de5923d7f836fe1f4 (commit) via 7ff1b945f6c8b2286f04626be39291a7527119f4 (commit) from 825b4ff35aa676b139dc24bc651724b092f2fded (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 9d1e8084dfa0a1ebae6c37e3b839289cb905116f Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jan 30 21:56:53 2012 -0600 Update translations Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 7b2f600d717ce5ecd0025d2fba43d405adb58bd4 Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jan 30 21:49:34 2012 -0600 Update pot translation catalogs Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 5c2928aa259cac1dcdaa8b27172c2ee12327d21a Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jan 30 21:43:23 2012 -0600 Make a few more copyright date tweaks Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 9e9835f4640a45cc758fa64de5923d7f836fe1f4 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Mon Jan 23 17:14:25 2012 -0500 scripts: always use printf with embedded gettext This addresses two problems: 1) echo's behavior is inconsistent when dealing with flags, and can potentially be problematic. $ echo -n $ echo -- -n -- -n 2) Always using the end of options markers prevents translated strings from throwing errors, as shown in FS#28069. The remaining "inconsistencies" are because printf is being used in a guaranteed safe manner, e.g. printf '%s\n' "$(gettext "--this can never break")" Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 7ff1b945f6c8b2286f04626be39291a7527119f4 Author: Allan McRae <allan(a)archlinux.org> Date: Mon Jan 30 11:47:48 2012 +1000 makepkg: workaround for zipman issues This "fixes" two issues: 1) MAN_DIRS contains a glob by default so should not be quoted. It is not quoted anywhere else so this should not cause breakage... 2) the find statement returns 1 when some of MAN_DIRS are missing. This appears to only be exposed when running makepkg as root (which it appears some wrappers do...). Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: doc/index.txt | 2 +- lib/libalpm/po/ru.po | 37 ++++--- scripts/makepkg.sh.in | 81 +++++++------- scripts/pacman-db-upgrade.sh.in | 8 +- scripts/pacman-key.sh.in | 50 +++++----- scripts/pacman-optimize.sh.in | 12 +- scripts/pkgdelta.sh.in | 8 +- scripts/po/ca.po | 96 ++++++++++++++--- scripts/po/cs.po | 96 ++++++++++++++--- scripts/po/da.po | 90 +++++++++++++---- scripts/po/de.po | 97 ++++++++++++++--- scripts/po/el.po | 102 ++++++++++++++---- scripts/po/en_GB.po | 94 ++++++++++++----- scripts/po/es.po | 96 ++++++++++++++--- scripts/po/fi.po | 95 ++++++++++++++--- scripts/po/fr.po | 107 +++++++++++++++---- scripts/po/hu.po | 110 +++++++++++++++----- scripts/po/it.po | 96 ++++++++++++++--- scripts/po/kk.po | 97 ++++++++++++++---- scripts/po/lt.po | 96 ++++++++++++++--- scripts/po/nb.po | 100 ++++++++++++++---- scripts/po/pacman-scripts.pot | 72 ++++++++++--- scripts/po/pl.po | 94 +++++++++++++--- scripts/po/pt.po | 98 ++++++++++++++---- scripts/po/pt_BR.po | 100 ++++++++++++++---- scripts/po/ro.po | 98 ++++++++++++++---- scripts/po/ru.po | 222 ++++++++++++++++++++++++++++----------- scripts/po/sk.po | 159 +++++++++++++++++++++------- scripts/po/sr.po | 96 ++++++++++++++--- scripts/po/sr(a)latin.po | 96 ++++++++++++++--- scripts/po/sv.po | 90 +++++++++++++---- scripts/po/tr.po | 98 ++++++++++++++---- scripts/po/uk.po | 96 ++++++++++++++--- scripts/po/zh_CN.po | 96 ++++++++++++++--- scripts/po/zh_TW.po | 76 +++++++++++--- scripts/repo-add.sh.in | 44 ++++---- src/pacman/po/ca.po | 6 +- src/pacman/po/cs.po | 6 +- src/pacman/po/da.po | 6 +- src/pacman/po/de.po | 6 +- src/pacman/po/el.po | 6 +- src/pacman/po/en_GB.po | 6 +- src/pacman/po/es.po | 6 +- src/pacman/po/fi.po | 6 +- src/pacman/po/fr.po | 6 +- src/pacman/po/hu.po | 10 +-- src/pacman/po/it.po | 6 +- src/pacman/po/kk.po | 6 +- src/pacman/po/lt.po | 6 +- src/pacman/po/nb.po | 6 +- src/pacman/po/pacman.pot | 6 +- src/pacman/po/pl.po | 6 +- src/pacman/po/pt.po | 6 +- src/pacman/po/pt_BR.po | 6 +- src/pacman/po/ro.po | 6 +- src/pacman/po/ru.po | 139 +++++++++++++------------ src/pacman/po/sk.po | 6 +- src/pacman/po/sr.po | 6 +- src/pacman/po/sr(a)latin.po | 6 +- src/pacman/po/sv.po | 6 +- src/pacman/po/tr.po | 16 +-- src/pacman/po/uk.po | 33 +++---- src/pacman/po/zh_CN.po | 6 +- src/pacman/po/zh_TW.po | 6 +- 64 files changed, 2509 insertions(+), 938 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH] makepkg: workaround for zipman issues
by Allan McRae 30 Jan '12

30 Jan '12

This "fixes" two issues: 1) MAN_DIRS contains a glob by default so should not be quoted. It is not quoted anywhere else so this should not cause extra breakage... 2) the find statement returns 1 when some of MAN_DIRS are missing. This appears to only be exposed when running makepkg as root (which it appears some wrappers do...). Signed-off-by: Allan McRae <allan(a)archlinux.org> --- This is for maint. Both "fixes" are crappy but addressing them properly will require much larger changes that are not suitable for the 4.0.2 release. scripts/makepkg.sh.in | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index f269fc0..fee6512 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -1003,7 +1003,8 @@ tidy_install() { # the '|| true' part keeps the script from bailing on the EOF returned # by read at the end of the find output IFS=$'\n' read -rd '' -a hardlinks < \ - <(find "${MAN_DIRS[@]}" \! -name "$file" -samefile "$manpage" 2>/dev/null) || true + <(find ${MAN_DIRS[@]} \! -name "$file" -samefile "$manpage" \ + 2>/dev/null || true) || true rm -f "${hardlinks[@]}" # compress the original gzip -9 "$manpage" -- 1.7.9

1 0

Re: [pacman-dev] [aur-general] makepkg/yaourt issues
by Allan McRae 25 Jan '12

25 Jan '12

On 25/01/12 09:45, masutu wrote: > (i) zipman option in "/etc/makepkg.conf" > > if zipman is enabled in "/etc/makepkg.conf", makepkg gives errors like > > [...] > ==> Tidying install... > -> Purging unwanted files... > -> Compressing man and info pages... > ==> ERROR: An unknown error has occurred. Exiting... > [...] > > when running as root (yaourt/makepkg). Surprisingly man pages do get > gzipped and work correctly, so no real problem, just wondering. When > running makepkg as normal user, this error message doesn't occur. Similar > issue is discussed in https://bbs.archlinux.org/viewtopic.php?pid=1046860. > I can replicate this issue only when running makepkg as root... So why does this line fail when running as root but not as a user: IFS=$'\n' read -rd '' -a hardlinks < \ <(find "${MAN_DIRS[@]}" \! -name "$file" -samefile "$manpage" 2>/dev/null) || true It is the find line that is causing this issue due to non-existent directories in MAN_DIRS. We used to have a "|| true" after the actual find command to stop that but it got removed when switching to using "read". Allan

1 0

[pacman-dev] [PATCH 1/2] add library support for tuning TCP_KEEPALIVE
by Dave Reisner 24 Jan '12

24 Jan '12

This adds three new options to control the behavior of TCP keepalives: - CURLOPT_TCP_KEEPALIVE: enable/disable probes - CURLOPT_TCP_KEEPIDLE: idle time before sending first probe - CURLOPT_TCP_KEEPINTVL: delay between successive probes While not all operating systems support the TCP_KEEPIDLE and TCP_KEEPINTVL knobs, the library will still allow these options to be set by clients, silently ignoring the values. --- Playing the role of the optimist and labelling this work for 7.24.1. This is mostly the result of a conversation I had with Daniel regarding usage of keepalives to thwart control connections to FTP servers being killed off by firewalls. The code is more or less copied from the front end tool, with all the bells and whistles added to wire it up for accessibility via curl_easy_setopt. docs/libcurl/curl_easy_setopt.3 | 18 ++++++++++++++++++ docs/libcurl/symbols-in-versions | 3 +++ include/curl/curl.h | 7 +++++++ lib/connect.c | 32 ++++++++++++++++++++++++++++++++ lib/url.c | 20 ++++++++++++++++++++ lib/urldata.h | 4 ++++ 6 files changed, 84 insertions(+), 0 deletions(-) diff --git a/docs/libcurl/curl_easy_setopt.3 b/docs/libcurl/curl_easy_setopt.3 index 1a20cb9..0a4fef3 100644 --- a/docs/libcurl/curl_easy_setopt.3 +++ b/docs/libcurl/curl_easy_setopt.3 @@ -915,6 +915,24 @@ overdone. .IP CURLOPT_ADDRESS_SCOPE Pass a long specifying the scope_id value to use when connecting to IPv6 link-local or site-local addresses. (Added in 7.19.0) + +.IP CURLOPT_TCP_KEEPALIVE +Pass a long. If set to 1, TCP keepalive probes will be sent. The delay and +frequency of these probes can be controlled by the \fICURLOPT_TCP_KEEPIDLE\fP +and \fICURLOPT_TCP_KEEPINTVL\fP options, provided the operating system supports +them. Set to 0 (default behavior) to disable keepalive probes (Added in +7.24.1). + +.IP CURLOPT_TCP_KEEPIDLE +Pass a long. Sets the delay, in seconds, that the operating system will wait +while the connection is idle before sending keepalive probes. Not all operating +systems support this option. (Added in 7.24.1) + +.IP CURLOPT_TCP_KEEPINTVL +Pass a long. Sets the interval, in seconds, that the operating system will wait +between sending keepalive probes. Not all operating systems support this +option. (Added in 7.24.1) + .SH NAMES and PASSWORDS OPTIONS (Authentication) .IP CURLOPT_NETRC This parameter controls the preference of libcurl between using user names and diff --git a/docs/libcurl/symbols-in-versions b/docs/libcurl/symbols-in-versions index 73d50a2..a9f8a9d 100644 --- a/docs/libcurl/symbols-in-versions +++ b/docs/libcurl/symbols-in-versions @@ -486,6 +486,9 @@ CURLOPT_SSL_SESSIONID_CACHE 7.16.0 CURLOPT_SSL_VERIFYHOST 7.8.1 CURLOPT_SSL_VERIFYPEER 7.4.2 CURLOPT_STDERR 7.1 +CURLOPT_TCP_KEEPALIVE 7.24.1 +CURLOPT_TCP_KEEPIDLE 7.24.1 +CURLOPT_TCP_KEEPINTVL 7.24.1 CURLOPT_TCP_NODELAY 7.11.2 CURLOPT_TELNETOPTIONS 7.7 CURLOPT_TFTP_BLKSIZE 7.19.4 diff --git a/include/curl/curl.h b/include/curl/curl.h index 59a5c79..2a9957e 100644 --- a/include/curl/curl.h +++ b/include/curl/curl.h @@ -1499,6 +1499,13 @@ typedef enum { of miliseconds. */ CINIT(ACCEPTTIMEOUT_MS, LONG, 212), + /* Set TCP keepalive */ + CINIT(TCP_KEEPALIVE, LONG, 213), + + /* non-universal keepalive knobs (Linux, AIX, HP-UX, more) */ + CINIT(TCP_KEEPIDLE, LONG, 214), + CINIT(TCP_KEEPINTVL, LONG, 215), + CURLOPT_LASTENTRY /* the last unused */ } CURLoption; diff --git a/lib/connect.c b/lib/connect.c index f76ec0e..f882db5 100644 --- a/lib/connect.c +++ b/lib/connect.c @@ -91,6 +91,35 @@ static bool verifyconnect(curl_socket_t sockfd, int *error); +static void +tcpkeepalive(struct SessionHandle *data, + int sockfd) +{ + int optval = data->set.tcp_keepalive; + + /* don't abort operation if any setsockopt fails, just log to debug */ + if(setsockopt(sockfd, SOL_SOCKET, SO_KEEPALIVE, + (void *)&optval, sizeof(optval)) < 0) { + infof(data, "Failed to set SO_KEEPALIVE on fd %d\n", sockfd); + } + else { +#ifdef TCP_KEEPIDLE + optval = data->set.tcp_keepidle; + if(setsockopt(sockfd, IPPROTO_TCP, TCP_KEEPIDLE, + (void *)&optval, sizeof(optval)) < 0) { + infof(data, "Failed to set TCP_KEEPIDLE on fd %d\n", sockfd); + } +#endif +#ifdef TCP_KEEPINTVL + optval = data->set.tcp_keepintvl; + if(setsockopt(sockfd, IPPROTO_TCP, TCP_KEEPINTVL, + (void *)&optval, sizeof(optval)) < 0) { + infof(data, "Failed to set TCP_KEEPINTVL on fd %d\n", sockfd); + } +#endif + } +} + static CURLcode singleipconnect(struct connectdata *conn, const Curl_addrinfo *ai, /* start connecting to this */ @@ -874,6 +903,9 @@ singleipconnect(struct connectdata *conn, Curl_sndbufset(sockfd); + if(data->set.tcp_keepalive) + tcpkeepalive(data, sockfd); + if(data->set.fsockopt) { /* activate callback for setting socket options */ error = data->set.fsockopt(data->set.sockopt_client, diff --git a/lib/url.c b/lib/url.c index 466748b..9d66e76 100644 --- a/lib/url.c +++ b/lib/url.c @@ -742,6 +742,13 @@ CURLcode Curl_init_userdefined(struct UserDefined *set) set->chunk_bgn = ZERO_NULL; set->chunk_end = ZERO_NULL; + /* tcp keepalives are disabled by default, but provide reasonable values for + * the interval and idle times. + */ + set->tcp_keepalive = 0; + set->tcp_keepintvl = 60; + set->tcp_keepidle = 60; + return res; } @@ -805,6 +812,7 @@ CURLcode Curl_open(struct SessionHandle **curl) multi stack. */ } + if(res) { Curl_resolver_cleanup(data->state.resolver); if(data->state.headerbuff) @@ -2539,6 +2547,18 @@ CURLcode Curl_setopt(struct SessionHandle *data, CURLoption option, result = Curl_set_dns_servers(data, va_arg(param, char *)); break; + case CURLOPT_TCP_KEEPALIVE: + data->set.tcp_keepalive = (0 != va_arg(param, long))?TRUE:FALSE; + break; + + case CURLOPT_TCP_KEEPIDLE: + data->set.tcp_keepidle = va_arg(param, long); + break; + + case CURLOPT_TCP_KEEPINTVL: + data->set.tcp_keepintvl = va_arg(param, long); + break; + default: /* unknown tag and its companion, just ignore: */ result = CURLE_UNKNOWN_OPTION; diff --git a/lib/urldata.h b/lib/urldata.h index adabf5b..a959bc7 100644 --- a/lib/urldata.h +++ b/lib/urldata.h @@ -1539,6 +1539,10 @@ struct UserDefined { long gssapi_delegation; /* GSSAPI credential delegation, see the documentation of CURLOPT_GSSAPI_DELEGATION */ + + bool tcp_keepalive; /* use TCP keepalives */ + long tcp_keepidle; /* seconds in idle before sending keepalive probe */ + long tcp_keepintvl; /* seconds between TCP keepalive probes */ }; struct Names { -- 1.7.8.4

1 1

[pacman-dev] [GIT] The official pacman repository branch, master, updated. v4.0.1-158-gbe038f9
by dan＠archlinux.org 23 Jan '12

23 Jan '12

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 be038f9cb2d4302d74c0edadd9abf2b282adc7c3 (commit) via ac239c54d0b8f83ee8a0a94456fc84da7f583ccb (commit) via e50c4a8837cedf47c2fe1a2c6064b4dc03dc5b03 (commit) via 825b4ff35aa676b139dc24bc651724b092f2fded (commit) via a03c35125ec729b914600b7c24084f83a05a5ede (commit) via 44f146f232be5203fb01ad35fdf73122838df97c (commit) via edd4276bbf3d21a7353e3d67ce6639246ef8032d (commit) via df47136bcfd3fdec150fc8613f3fe243432d875f (commit) via de56874cc9cdbfa96dd6360818ee6f21da3e1c86 (commit) via 1eb40c83287b07ac7428ad2d58504f386fad98f1 (commit) via 562109c0e8717eaac3b9078271c4ca4f82abfecd (commit) via b3612e9cc198f198b3806efa461bf6fc04dd4502 (commit) via 1b50223f8240456b8c989b5c1e016f4a245b527c (commit) via c77cec2ffc850fa28c5720b8902acc5421069ae4 (commit) via c231c9af9712e95e58f660d46bd8feaf6fd891e2 (commit) via 24ca6ce1f969a6f5d3ef9277f6d20efcd76330ec (commit) via 430b0df7794815049f37f38df39c71e1a9e9c157 (commit) via d9af1a0cf2383eed009ed2bedfb1b34f1a5c7418 (commit) via 4e60b9646d69227c6c9f226bf03a12158feaaee7 (commit) via b426488e2b1cf4e464f7f56dbcf2629a3a73a54a (commit) via 7b1a86b8939f59693f8bb0ea0454a702d5e2434e (commit) via be229d129eeb43e774217921e1c7e1bb802775fe (commit) via e8a2c2545608c556cb65d96e7015a1d09d8b3890 (commit) via 6e8ca48cbb22c328deab56b1740be04ea7ddba6e (commit) via b6209b4ba495dd4379806b72735dac78b8843c03 (commit) from 4f02b98338437b9395addb29000893b91d1ae21e (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 be038f9cb2d4302d74c0edadd9abf2b282adc7c3 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 21:43:00 2012 -0600 Remove rmrf implementation from backend This moves the code for removal of local database entries right into be_local.c, which was the last user of the rmrf() function we had in our utility source file. We can simplify the implementation and make it non-recursive as we know the structure of the local database entries. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit ac239c54d0b8f83ee8a0a94456fc84da7f583ccb Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 22:09:43 2012 -0600 libalpm/deps.c: access trans flags directly Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e50c4a8837cedf47c2fe1a2c6064b4dc03dc5b03 Merge: b3612e9 825b4ff Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jan 23 12:19:51 2012 -0600 Merge branch 'maint' Conflicts: lib/libalpm/diskspace.c src/pacman/util.h commit b3612e9cc198f198b3806efa461bf6fc04dd4502 Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 11 15:39:52 2012 -0600 Allow UseDelta option to specify a delta ratio Rework the frontend and backend to allow passing a ratio value in for UseDelta rather than having a hardcoded #define-d 0.7 value always used. This is useful for those with fast connections, who would likely benefit from tuning this ratio to lower values; it is also useful for general testing purposes. The libalpm API changes for this, but we do support the old config file format with a no-value 'UseDelta' option; in this case we simply use the old default of 0.7. We clamp the ratio values to a sane range between 0.0 and 2.0, allowing ratios above 1.0 for testing purposes. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 6e8ca48cbb22c328deab56b1740be04ea7ddba6e Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 11 12:04:34 2012 -0600 Reorder some operations in sig check for efficiency We don't need to open the data to be checked if we don't have a signature to check against, so postpone that open until we know we have either the base64_data or a valid signature file. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: configure.ac | 4 +- contrib/bash_completion.in | 4 +- contrib/zsh_completion.in | 2 +- doc/pacman.8.txt | 2 +- doc/pacman.conf.5.txt | 10 ++- doc/vercmp.8.txt | 9 ++- etc/pacman.conf.in | 2 +- lib/libalpm/alpm.h | 4 +- lib/libalpm/be_local.c | 40 +++++++++- lib/libalpm/delta.c | 3 +- lib/libalpm/delta.h | 3 - lib/libalpm/deps.c | 6 +- lib/libalpm/diskspace.c | 56 +++++++++++++- lib/libalpm/dload.c | 155 ++++++++++++++++++++++++++++++--------- lib/libalpm/handle.c | 12 ++- lib/libalpm/handle.h | 2 +- lib/libalpm/remove.c | 2 +- lib/libalpm/signing.c | 22 ++---- lib/libalpm/sync.c | 4 +- lib/libalpm/util.c | 50 +------------ lib/libalpm/util.h | 2 +- scripts/makepkg.sh.in | 17 +++-- scripts/pacman-key.sh.in | 174 ++++++++++++++++++++++++++++++++++++-------- scripts/repo-add.sh.in | 24 ++++-- src/pacman/conf.c | 19 ++++- src/pacman/conf.h | 2 +- src/pacman/pacman.c | 4 +- src/pacman/sync.c | 34 +++------ src/pacman/util.c | 33 ++++----- src/pacman/util.h | 3 +- 30 files changed, 474 insertions(+), 230 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v4.0.1-45-g825b4ff
by dan＠archlinux.org 23 Jan '12

23 Jan '12

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, maint has been updated via 825b4ff35aa676b139dc24bc651724b092f2fded (commit) via a03c35125ec729b914600b7c24084f83a05a5ede (commit) via 44f146f232be5203fb01ad35fdf73122838df97c (commit) from edd4276bbf3d21a7353e3d67ce6639246ef8032d (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 825b4ff35aa676b139dc24bc651724b092f2fded Author: Dave Reisner <dreisner(a)archlinux.org> Date: Sun Jan 22 18:46:23 2012 -0500 lib/dload: give uniform naming to curl CB functions Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit a03c35125ec729b914600b7c24084f83a05a5ede Author: Dan McGee <dan(a)archlinux.org> Date: Mon Jan 23 12:15:53 2012 -0600 Fix sys/mnttab.h header include Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 44f146f232be5203fb01ad35fdf73122838df97c Author: Dave Reisner <dreisner(a)archlinux.org> Date: Sat Jan 21 21:31:39 2012 -0500 lib/dload: enforce usage of TCP keepalives This is particularly important in the case of FTP control connections, which may be closed by rogue NAT/firewall devices detecting idle connections on larger transfers which may take 5-10+ minutes. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: configure.ac | 3 +- lib/libalpm/diskspace.c | 4 +- lib/libalpm/dload.c | 61 +++++++++++++++++++++++++++++++++++++++++++--- 3 files changed, 61 insertions(+), 7 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [GIT] The official pacman repository branch, maint, updated. v4.0.1-42-gedd4276
by dan＠archlinux.org 23 Jan '12

23 Jan '12

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, maint has been updated via edd4276bbf3d21a7353e3d67ce6639246ef8032d (commit) via df47136bcfd3fdec150fc8613f3fe243432d875f (commit) via de56874cc9cdbfa96dd6360818ee6f21da3e1c86 (commit) via 1eb40c83287b07ac7428ad2d58504f386fad98f1 (commit) via 562109c0e8717eaac3b9078271c4ca4f82abfecd (commit) via 1b50223f8240456b8c989b5c1e016f4a245b527c (commit) via c77cec2ffc850fa28c5720b8902acc5421069ae4 (commit) via c231c9af9712e95e58f660d46bd8feaf6fd891e2 (commit) via 24ca6ce1f969a6f5d3ef9277f6d20efcd76330ec (commit) via 430b0df7794815049f37f38df39c71e1a9e9c157 (commit) via d9af1a0cf2383eed009ed2bedfb1b34f1a5c7418 (commit) via 4e60b9646d69227c6c9f226bf03a12158feaaee7 (commit) via b426488e2b1cf4e464f7f56dbcf2629a3a73a54a (commit) via 7b1a86b8939f59693f8bb0ea0454a702d5e2434e (commit) via be229d129eeb43e774217921e1c7e1bb802775fe (commit) via e8a2c2545608c556cb65d96e7015a1d09d8b3890 (commit) via b6209b4ba495dd4379806b72735dac78b8843c03 (commit) from 73d0d743bda5367fcab2453bbe21c15e481150c2 (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 edd4276bbf3d21a7353e3d67ce6639246ef8032d Author: Allan McRae <allan(a)archlinux.org> Date: Fri Jan 20 23:25:57 2012 +1000 makepkg: restrict usage of flags passed to pacman With pacman-4.0, using --noconfirm or --noprogressbar with -Q or -T results in pacman reporting an "invalid option" error. Restrict the passing of these options to pacman. Fixes FS#28012. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit df47136bcfd3fdec150fc8613f3fe243432d875f Author: Allan McRae <allan(a)archlinux.org> Date: Fri Jan 20 23:20:52 2012 +1000 makepkg: fix error on unnecessary -r The grep statement used to check for a difference between the installed package list before and after resolving dependencies returns 1 if there is no difference. This sets of the error trap when "-r" is used "unnecessarily". Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit de56874cc9cdbfa96dd6360818ee6f21da3e1c86 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jan 19 16:23:57 2012 -0600 contrib/bash_completion: don't print stderr messages If you are a crazy developer like me and have bogus options in your pacman.conf file, the tab completion gets messed up by the output on stderr. Suppress it. Fix the same basic issue in zsh_completion, thanks to the work by Florian Pritz <bluewind(a)xinu.at>. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1eb40c83287b07ac7428ad2d58504f386fad98f1 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jan 19 16:21:06 2012 -0600 Add diskspace checking support for Solaris/Illumos Was able to get my hands on one of these boxes today, so add yet another new way of doing this. I'm glad these calls are so standardized. This was compile tested on Linux and Illumos and seems to still be working in both places. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 562109c0e8717eaac3b9078271c4ca4f82abfecd Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 22:25:27 2012 -0600 Update copyright on changed files since beginning of year Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 1b50223f8240456b8c989b5c1e016f4a245b527c Author: Olivier Brunel <i.am.jack.mail(a)gmail.com> Date: Mon Jan 9 21:37:31 2012 +0100 util.c, rmrf(): only create string when needed The entry's name is only used when not "." or ".." so only print the string then. Signed-off-by: Olivier Brunel <i.am.jack.mail(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c77cec2ffc850fa28c5720b8902acc5421069ae4 Author: Allan McRae <allan(a)archlinux.org> Date: Thu Jan 19 11:29:36 2012 +1000 Fix missing [removal] output Currently, a transaction is considered to be purely package removal until the first package install is found. This resulted in the removed packages at the start of a combined upgrade/removal transaction not getting the "[removal]" output. Fixes FS#27981. Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit c231c9af9712e95e58f660d46bd8feaf6fd891e2 Author: canyonknight <canyonknight(a)gmail.com> Date: Mon Jan 16 19:12:41 2012 -0500 Improve exit statuses and error messages in pacman-key Return codes from gpg commands are currently lost. This adds the functionality of taking non-zero exit statuses from gpg. This includes error reporting for all gpg commands that are run individually, run in a loop, and run through a pipe. Includes the check_keyids_exist function which verifies a key exists locally prior to attempted local manipulation of the key. If a gpg command has a non-zero status, pacman-key will now exit with a non-zero status. It will print a gettext error message of gpg's failure. Signed-off-by: canyonknight <canyonknight(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 24ca6ce1f969a6f5d3ef9277f6d20efcd76330ec Author: canyonknight <canyonknight(a)gmail.com> Date: Mon Jan 16 19:09:56 2012 -0500 Turn gpg commands into functions in pacman-key Adds functions for every gpg command. By pulling out the gpg commands from the "program start" section, additional commands can be run before or after a specific gpg command without adding additional clutter to the function call section. Adds an explicit exit status of 0 to prevent arithmetic expansions from returning non-zero, thereby falsely causing pacman-key to have a non-zero exit status. This change creates the framework for additional error messages and better exit statuses being added to every pacman-key gpg call. Signed-off-by: canyonknight <canyonknight(a)gmail.com> Signed-off-by: Allan McRae <allan(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 430b0df7794815049f37f38df39c71e1a9e9c157 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jan 12 09:32:19 2012 -0600 repo-add: clean up help messages Use consistent blank lines across all commands, get rid of the translated double newlines which only serve to confuse translators, and fix -h/--help for that extra special third command this script offers. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit d9af1a0cf2383eed009ed2bedfb1b34f1a5c7418 Author: Olivier Brunel <i.am.jack.mail(a)gmail.com> Date: Thu Jan 12 16:08:48 2012 +0100 Fix broken output when asking question and stdin is piped When asking question and stdin is piped, the response does not get printed out, resulting in a missing \n and broken output (FS#27909); printing the response fixes it. Signed-off-by: Olivier Brunel <i.am.jack.mail(a)gmail.com> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 4e60b9646d69227c6c9f226bf03a12158feaaee7 Author: Dave Reisner <dreisner(a)archlinux.org> Date: Sat Jan 14 15:27:48 2012 -0500 fetch_url: look for files in cache before downloading We lost this logic somewhere between the libfetch and libcurl transition, as it existed in the internal downloader, but was pulled back only into the sync workflow. Add a helper function that will let us check for existance in the filecache prior to calling the downloader. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b426488e2b1cf4e464f7f56dbcf2629a3a73a54a Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 15:32:48 2012 -0600 Use isdigit() rather than character range comparisons This is safer and guaranteed to work with even exotic character sets. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit 7b1a86b8939f59693f8bb0ea0454a702d5e2434e Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 15:32:34 2012 -0600 Remove unused strtoupper() function Signed-off-by: Dan McGee <dan(a)archlinux.org> commit be229d129eeb43e774217921e1c7e1bb802775fe Author: Dan McGee <dan(a)archlinux.org> Date: Wed Jan 18 12:12:39 2012 -0600 Don't remove unknown files in cache cleaning code This removes the hack I added to skip '*.sig' files earlier since there are other files that also fall into the same bucket- source packages from `makepkg --source`, delta files, etc. Rather than prompting for each and every one, simply skip them. Doing '-Scc' rather than '-Sc' will delete these files if that is really what you want to do. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit e8a2c2545608c556cb65d96e7015a1d09d8b3890 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jan 12 17:29:14 2012 -0600 doc/vercmp: add note about pkgrel handling This comes from the Doxygen function documentation. Also, fix two rather silly misspellings. Signed-off-by: Dan McGee <dan(a)archlinux.org> commit b6209b4ba495dd4379806b72735dac78b8843c03 Author: Dan McGee <dan(a)archlinux.org> Date: Thu Jan 12 07:47:29 2012 -0600 Use fileno() in isatty() call This was our only use of the function that had a hardcoded file descriptor. Signed-off-by: Dan McGee <dan(a)archlinux.org> ----------------------------------------------------------------------- Summary of changes: configure.ac | 3 +- contrib/bash_completion.in | 4 +- contrib/zsh_completion.in | 2 +- doc/pacman.8.txt | 2 +- doc/vercmp.8.txt | 9 ++- lib/libalpm/diskspace.c | 54 ++++++++++++- lib/libalpm/dload.c | 94 +++++++++++++++-------- lib/libalpm/remove.c | 2 +- lib/libalpm/signing.c | 2 +- lib/libalpm/util.c | 10 +- scripts/makepkg.sh.in | 17 +++-- scripts/pacman-key.sh.in | 174 ++++++++++++++++++++++++++++++++++++-------- scripts/repo-add.sh.in | 24 ++++-- src/pacman/pacman.c | 4 +- src/pacman/sync.c | 34 +++------ src/pacman/util.c | 33 ++++----- src/pacman/util.h | 3 +- 17 files changed, 330 insertions(+), 141 deletions(-) hooks/post-receive -- The official pacman repository

1 0

[pacman-dev] [PATCH 1/2] lib/dload: enforce usage of TCP keepalives
by Dave Reisner 23 Jan '12

23 Jan '12

This is particularly important in the case of FTP control connections, which may be closed by rogue NAT/firewall devices detecting idle connections on larger transfers which may take 5-10+ minutes. Signed-off-by: Dave Reisner <dreisner(a)archlinux.org> --- This is basically on the advice of Daniel Stenberg, who mentions that it's a common problem that he's aware of. I have no idea if this is actually the fix we're looking for and I'm still unable to reproduce, but it seems plausible solution. Also, fuck FTP. configure.ac | 7 +++-- lib/libalpm/dload.c | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 63 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 6941054..c8ba03f 100644 --- a/configure.ac +++ b/configure.ac @@ -174,9 +174,10 @@ AM_CONDITIONAL([HAVE_LIBGPGME], [test "x$with_gpgme" = "xyes"]) # Checks for header files. AC_CHECK_HEADERS([fcntl.h float.h glob.h libintl.h limits.h locale.h \ - mntent.h stddef.h string.h sys/ioctl.h sys/mount.h \ - sys/param.h sys/statvfs.h sys/time.h sys/types.h \ - sys/ucred.h syslog.h termios.h wchar.h]) + mntent.h netinet/in.h netinet/tcp.h stddef.h string.h \ + sys/ioctl.h sys/mount.h sys/param.h sys/statvfs.h \ + sys/time.h sys/types.h sys/ucred.h syslog.h termios.h \ + wchar.h]) # Checks for typedefs, structures, and compiler characteristics. AC_C_INLINE diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index a3980d6..790080c 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -23,11 +23,19 @@ #include <errno.h> #include <string.h> #include <unistd.h> +#include <sys/socket.h> /* setsockopt, SO_KEEPALIVE */ #include <sys/time.h> #include <sys/types.h> #include <sys/stat.h> #include <signal.h> +#ifdef _HAVE_NETINET_IN_H +#include <netinet/in.h> /* IPPROTO_TCP */ +#endif +#ifdef _HAVE_NETINET_TCP_H +#include <netinet/tcp.h> /* TCP_KEEPINTVL, TCP_KEEPIDLE */ +#endif + #ifdef HAVE_LIBCURL #include <curl/curl.h> #endif @@ -215,6 +223,53 @@ static size_t parse_headers(void *ptr, size_t size, size_t nmemb, void *user) return realsize; } +static int curl_sockopt_cb(void *userdata, curl_socket_t curlfd, + curlsocktype purpose) +{ + alpm_handle_t *handle = userdata; + + int onoff = 1; + +#ifdef TCP_KEEPIDLE + int keepidle = TCP_KEEPIDLE; +#endif +#ifdef TCP_KEEPINTVL + int keepintvl = TCP_KEEPINTVL; +#endif + + switch(purpose) { + case CURLSOCKTYPE_IPCXN: + if(setsockopt(curlfd, SOL_SOCKET, SO_KEEPALIVE, (void *)&onoff, + sizeof(onoff)) < 0) { + /* don't abort operation, just log to debug */ + _alpm_log(handle, ALPM_LOG_DEBUG, "Failed to set SO_KEEPALIVE on fd %d\n", + curlfd); + } + else { +#ifdef TCP_KEEPIDLE + if(setsockopt(curlfd, IPPROTO_TCP, TCP_KEEPIDLE, (void *)&keepidle, + sizeof(keepidle)) < 0) { + /* don't abort operation, just log to debug */ + _alpm_log(handle, ALPM_LOG_DEBUG, "Failed to set TCP_KEEPIDLE on fd %d\n", + curlfd); + } +#endif +#ifdef TCP_KEEPINTVL + if(setsockopt(curlfd, IPPROTO_TCP, TCP_KEEPINTVL, (void *)&keepintvl, + sizeof(keepintvl)) < 0) { + /* don't abort operation, just log to debug */ + _alpm_log(handle, ALPM_LOG_DEBUG, "Failed to set TCP_KEEPINTVL on fd %d\n", + curlfd); + } +#endif + } + default: + break; + } + + return 0; +} + static void curl_set_handle_opts(struct dload_payload *payload, CURL *curl, char *error_buffer) { @@ -239,6 +294,8 @@ static void curl_set_handle_opts(struct dload_payload *payload, curl_easy_setopt(curl, CURLOPT_HEADERFUNCTION, parse_headers); curl_easy_setopt(curl, CURLOPT_WRITEHEADER, (void *)payload); curl_easy_setopt(curl, CURLOPT_NETRC, CURL_NETRC_OPTIONAL); + curl_easy_setopt(curl, CURLOPT_SOCKOPTFUNCTION, curl_sockopt_cb); + curl_easy_setopt(curl, CURLOPT_SOCKOPTDATA, (void *)handle); _alpm_log(handle, ALPM_LOG_DEBUG, "url: %s\n", payload->fileurl); @@ -388,6 +445,8 @@ static int curl_download_internal(struct dload_payload *payload, /* perform transfer */ payload->curlerr = curl_easy_perform(curl); + _alpm_log(handle, ALPM_LOG_DEBUG, "curl returned error %d from transfer\n", + payload->curlerr); /* disconnect relationships from the curl handle for things that might go out * of scope, but could still be touched on connection teardown. This really -- 1.7.8.4

3 5

[pacman-dev] [PATCH] makepkg: fix error on unnecessary -r
by Allan McRae 23 Jan '12

23 Jan '12

The grep statement used to check for a difference between the installed package list before and after resolving dependencies returns 1 if there is no difference. This sets of the error trap when "-r" is used "unnecessarily". Signed-off-by: Allan McRae <allan(a)archlinux.org> --- Dave: would you prefer this fix or just using $(set +E; grep ...) scripts/makepkg.sh.in | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in index f6d8294..4792c5c 100644 --- a/scripts/makepkg.sh.in +++ b/scripts/makepkg.sh.in @@ -507,14 +507,15 @@ remove_deps() { # check for packages removed during dependency install (e.g. due to conflicts) # removing all installed packages is risky in this case if [[ -n $(grep -xvFf <(printf '%s\n' "${current_packagelist[@]}") \ - <(printf '%s\n' "${original_packagelist[@]}") ) ]]; then + <(printf '%s\n' "${original_packagelist[@]}") || true) ]]; then warning "$(gettext "Failed to remove installed dependencies.")" return 0 fi local deplist - if ! deplist=($(grep -xvFf <(printf "%s\n" "${original_pkglist[@]}") \ - <(printf "%s\n" "${current_pkglist[@]}"))); then + deplist=($(grep -xvFf <(printf "%s\n" "${original_pkglist[@]}") \ + <(printf "%s\n" "${current_pkglist[@]}") || true)) + if [[ -n deplist ]]; then return fi -- 1.7.8.4

2 5

[pacman-dev] compiling pacman and python wrapper
by Andrea Crotti 22 Jan '12

22 Jan '12

I would like to create a python wrapper to access to pacman features, and so I started to take a look about how it works.. The first thing that I noted is that the INSTALL instructions are a bit confusing. Because it says: --8<---------------cut here---------------start------------->8--- Briefly, the shell commands `./configure; make; make install' should configure, build, and install this package. The following instructions are generic. Run `./configure --help` for specific options. --8<---------------cut here---------------end--------------->8--- But actually there is no "configure" until autoconf is run, but that is explained only later. So maybe just inverting the order might be fine, first talking about autoconf and configure.ac and after talking about configure && make. Then I was actually trying to compile it myself but I get on configure: --8<---------------cut here---------------start------------->8--- configure: creating ./config.status config.status: error: cannot find input file: `lib/libalpm/Makefile.in' --8<---------------cut here---------------end--------------->8--- (from the git repository). And about my original idea, is anyone interested? From what I've understood I think I might use SWIG using the file alpm.h as header and export all the symbols in a nice Python wrapper, does it make sense?

4 6