September 2015 - Aur-dev - lists.archlinux.org

[aur-dev] Setting keywords from the command line
by Lukas Fleischer 16 Oct '15

16 Oct '15

Hi, Currently, in order to change the keywords of a package base, users need to log into the web interface and submit a form on the package base details page. It would be nice to be able to change keywords without using a web browser, see FS#45627 [1]. Now, there are two options: 1. Add a set-keywords command to the aurweb SSH interface such that users can run `ssh aur.archlinux.org set-keywords foobar keyword1 keyword2 keyword3` to change keywords (or still use the web interface). 2. Remove the option from the web interface and introduce a new file (.KEYWORDS or something more general like .AURINFO, .AURMETADATA) that is parsed in the Git update hook. Obviously, with the second option, one would get a history of keyword changes. On the other hand, some might argue that this will clutter the commit history (although keyword changes should not happen too often) and that this changes makes the AUR package repositories special in a certain way (currently there is nothing AUR-specific in the Git repositories itself, an implementation of the second idea will add such information). Are there any opinions on which option should be implemented? Any other advantages/disadvantages I did not think of? Regards, Lukas [1] https://bugs.archlinux.org/task/45627

3 2

[aur-dev] [PATCH] Show a warning if .SRCINFO is unchanged
by Lukas Fleischer 29 Sep '15

29 Sep '15

Warn users when a remote ref update does not change the content of .SRCINFO such that users are reminded of updating package meta data. Implements FS#46130. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- git-interface/git-update.py | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/git-interface/git-update.py b/git-interface/git-update.py index 4698382..b418525 100755 --- a/git-interface/git-update.py +++ b/git-interface/git-update.py @@ -169,6 +169,9 @@ def die(msg): sys.stderr.write("error: {:s}\n".format(msg)) exit(1) +def warn(msg): + sys.stderr.write("warning: {:s}\n".format(msg)) + def die_commit(msg, commit): sys.stderr.write("error: The following error " + "occurred when parsing commit\n") @@ -282,6 +285,13 @@ for commit in walker: if not fname in commit.tree: die_commit('missing source file: {:s}'.format(fname), str(commit.id)) +# Display a warning if .SRCINFO is unchanged. +if sha1_old != "0000000000000000000000000000000000000000": + srcinfo_id_old = repo[sha1_old].tree['.SRCINFO'].id + srcinfo_id_new = repo[sha1_new].tree['.SRCINFO'].id + if srcinfo_id_old == srcinfo_id_new: + warn(".SRCINFO unchanged. The package database will not be updated!") + # Read .SRCINFO from the HEAD commit. srcinfo_raw = repo[repo[sha1_new].tree['.SRCINFO'].id].data.decode() srcinfo_raw = srcinfo_raw.split('\n') -- 2.6.0

1 0

[aur-dev] [PATCH] Remove empty package bases after 24 hours
by Lukas Fleischer 29 Sep '15

29 Sep '15

By using the setup-repo command, it is currently possible to create empty package bases, which can be used to make package base reservations. Add a maintenance script to remove such empty package bases after 24 hours. Fixes FS#46279. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- scripts/pkgmaint.py | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100755 scripts/pkgmaint.py diff --git a/scripts/pkgmaint.py b/scripts/pkgmaint.py new file mode 100755 index 0000000..0eb9422 --- /dev/null +++ b/scripts/pkgmaint.py @@ -0,0 +1,25 @@ +#!/usr/bin/python3 + +import configparser +import mysql.connector +import os + +config = configparser.RawConfigParser() +config.read(os.path.dirname(os.path.realpath(__file__)) + "/../conf/config") + +aur_db_host = config.get('database', 'host') +aur_db_name = config.get('database', 'name') +aur_db_user = config.get('database', 'user') +aur_db_pass = config.get('database', 'password') +aur_db_socket = config.get('database', 'socket') + +db = mysql.connector.connect(host=aur_db_host, user=aur_db_user, + passwd=aur_db_pass, db=aur_db_name, + unix_socket=aur_db_socket, buffered=True) +cur = db.cursor() + +cur.execute("DELETE FROM PackageBases WHERE " + + "UNIX_TIMESTAMP() - SubmittedTS > 86400 AND PackagerUID IS NULL") + +db.commit() +db.close() -- 2.6.0

1 0

[aur-dev] [PATCH] AUTHORS: Add date ranges to current maintainers
by Lukas Fleischer 29 Sep '15

29 Sep '15

Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- AUTHORS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/AUTHORS b/AUTHORS index 4462cab..18c1337 100644 --- a/AUTHORS +++ b/AUTHORS @@ -1,8 +1,8 @@ Current Maintainers ------------------- -* Lukas Fleischer <lfleischer(a)archlinux.org> -* Johannes Löthberg <johannes(a)kyriasis.com> +* Lukas Fleischer <lfleischer(a)archlinux.org> (2011-present) +* Johannes Löthberg <johannes(a)kyriasis.com> (2015-present) Past Maintainers ---------------- -- 2.6.0

1 0

[aur-dev] 4.1.0 string freeze
by Lukas Fleischer 26 Sep '15

26 Sep '15

Dear translators, I just submitted the final message catalog for 4.1.0. There are 26 new strings to translate. The release is scheduled for Saturday, October 3rd. Thank you for your continued contributions! Regards, Lukas

1 0

[aur-dev] Translation-String cleanup
by Christoph Seitz 26 Sep '15

26 Sep '15

Hello there, I just translated the new strings. I found some String pairs having the same meaning but different wording. Example: " You are not allowed to edit this comment." and "You do not have the right to edit this comment." Maybe we should do some clean up in order to keep the string base handy. Would you accept some patches in this direction? Maybe there are only a few. Regards, Chris

2 2

[aur-dev] [PATCH] aurjson.class.php: Sync error message with front-end
by Lukas Fleischer 26 Sep '15

26 Sep '15

Instead of introducing a new message "You do not have the right to edit this comment." for the RPC interface, use "You are not allowed to edit this comment." which we already show in the front-end. Reported-by: Christoph Seitz <seitz.christoph(a)gmail.com> Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/lib/aurjson.class.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php index 304a50a..db9cc54 100644 --- a/web/lib/aurjson.class.php +++ b/web/lib/aurjson.class.php @@ -532,7 +532,7 @@ class AurJSON { if (!has_credential(CRED_COMMENT_EDIT, array($user_id))) { $output = array( 'success' => 0, - 'error' => __('You do not have the right to edit this comment.') + 'error' => __('You are not allowed to edit this comment.') ); return json_encode($output); } elseif (is_null($comment)) { -- 2.5.3

1 0

[aur-dev] [PATCH v2 1/2] aurjson.class.php: Fix "Undefined index" notices
by Lukas Fleischer 26 Sep '15

26 Sep '15

Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- Use isset() instead of array_key_exists() as suggested by Florian. web/lib/aurjson.class.php | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php index e646c63..304a50a 100644 --- a/web/lib/aurjson.class.php +++ b/web/lib/aurjson.class.php @@ -110,8 +110,8 @@ class AurJSON { return; } - $callback = $http_data['callback']; - if (isset($callback)) { + if (isset($http_data['callback'])) { + $callback = $http_data['callback']; if (!preg_match('/^[a-zA-Z0-9().]{1,128}$/D', $callback)) { return $this->json_error('Invalid callback name.'); } @@ -281,11 +281,15 @@ class AurJSON { * proper data types in the JSON response. */ foreach (self::$numeric_fields as $field) { - $row[$field] = intval($row[$field]); + if (isset($row[$field])) { + $row[$field] = intval($row[$field]); + } } foreach (self::$decimal_fields as $field) { - $row[$field] = floatval($row[$field]); + if (isset($row[$field])) { + $row[$field] = floatval($row[$field]); + } } if ($this->version >= 2 && ($type == 'info' || $type == 'multiinfo')) { -- 2.5.3

1 1

[aur-dev] [PATCH 1/2] aurjson.class.php: Fix "Undefined index" notices
by Lukas Fleischer 24 Sep '15

24 Sep '15

Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/lib/aurjson.class.php | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/web/lib/aurjson.class.php b/web/lib/aurjson.class.php index e646c63..dd3c0f0 100644 --- a/web/lib/aurjson.class.php +++ b/web/lib/aurjson.class.php @@ -110,8 +110,8 @@ class AurJSON { return; } - $callback = $http_data['callback']; - if (isset($callback)) { + if (array_key_exists('callback', $http_data)) { + $callback = $http_data['callback']; if (!preg_match('/^[a-zA-Z0-9().]{1,128}$/D', $callback)) { return $this->json_error('Invalid callback name.'); } @@ -281,11 +281,15 @@ class AurJSON { * proper data types in the JSON response. */ foreach (self::$numeric_fields as $field) { - $row[$field] = intval($row[$field]); + if (array_key_exists($field, $row)) { + $row[$field] = intval($row[$field]); + } } foreach (self::$decimal_fields as $field) { - $row[$field] = floatval($row[$field]); + if (array_key_exists($field, $row)) { + $row[$field] = floatval($row[$field]); + } } if ($this->version >= 2 && ($type == 'info' || $type == 'multiinfo')) { -- 2.5.3

1 1

[aur-dev] [PATCH] Make it more clear that the bug tracker is for aurweb only
by Marcel Korpel 24 Sep '15

24 Sep '15

Signed-off-by: Marcel Korpel <marcel.korpel(a)gmail.com> --- web/html/home.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/html/home.php b/web/html/home.php index 7e9a00f..475370b 100644 --- a/web/html/home.php +++ b/web/html/home.php @@ -112,7 +112,7 @@ html_header( __("Home") ); <p> <?php echo __( - 'If you find a bug in the AUR web interface, please fill out a bug report on our %sbug tracker%s. Use the tracker to report bugs in the AUR %sonly%s. To report packaging bugs contact the package maintainer or leave a comment on the appropriate package page.', + 'If you find a bug in the AUR web interface, please fill out a bug report on our %sbug tracker%s. Use the tracker to report bugs in the AUR web interface %sonly%s. To report packaging bugs contact the package maintainer or leave a comment on the appropriate package page.', '<a href="https://bugs.archlinux.org/index.php?project=2">', '</a>', '<strong>', -- 2.5.3

2 1