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

[aur-dev] Display of alternative source name instead of URL on packages' web pages
by Peter Mattern 31 Oct '15

31 Oct '15

Hello. An alternative source name set in variable 'source' of PKBDUILD is replacing the URL in packages' web pages, see e. g. https://aur.archlinux.org/packages/cantata-git/ where section "Sources (1)" states "cantata-git" instead of "git+https://github.com/CDrummond/cantata.git". I'm not absolutely sure whether this behaviour may eventually be expected but actually don't think it makes a lot of sense. So wouldn't it be better to have the URL displayed regardless of an alternative source name is set or not? Regards, Peter Mattern

5 7

[aur-dev] [PATCH] notify: Fix merging of header dicts
by Lukas Fleischer 26 Oct '15

26 Oct '15

Fixes another regression introduced in c764999 (notify: Split out email header generation, 2015-10-10). Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- scripts/notify.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/scripts/notify.py b/scripts/notify.py index 4832baf..d3d9cb0 100755 --- a/scripts/notify.py +++ b/scripts/notify.py @@ -242,7 +242,8 @@ def request_open(cur, uid, reqid, reqtype, pkgbase_id, merge_into=None): refs = '[1] ' + user_uri + '\n' refs += '[2] ' + pkgbase_uri + '\n' thread_id = '<pkg-request-' + reqid + '@aur.archlinux.org>' - headers = headers_msgid(thread_id) + headers_cc(cc) + headers = headers_reply(thread_id) + headers.update(headers_cc(cc)) send_notification(to, subject, body, refs, headers) @@ -263,7 +264,8 @@ def request_close(cur, uid, reqid, reason): body += ':\n\n' + text refs = '[1] ' + user_uri thread_id = '<pkg-request-' + reqid + '@aur.archlinux.org>' - headers = headers_reply(thread_id) + headers_cc(cc) + headers = headers_reply(thread_id) + headers.update(headers_cc(cc)) send_notification(to, subject, body, refs, headers) -- 2.6.2

1 0

[aur-dev] [PATCH] footer: Close short open tag
by Lukas Fleischer 26 Oct '15

26 Oct '15

Fixes a regression introduced in fd6ba76 (Make copyright notice translatable, 2015-10-17). Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/template/footer.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/template/footer.php b/web/template/footer.php index 7e505a3..f5dc2d0 100644 --- a/web/template/footer.php +++ b/web/template/footer.php @@ -5,7 +5,7 @@ <?php if ($ver): ?> <p>aurweb <a href="https://projects.archlinux.org/aurweb.git/log/?h=<?= htmlspecialchars($ver, ENT_QUOTES) ?>"><?= htmlspecialchars($ver) ?></a></p> <?php endif; ?> - <p><?= __('Copyright %s 2004-%d aurweb Development Team.', '©', date('Y'))</p> + <p><?= __('Copyright %s 2004-%d aurweb Development Team.', '©', date('Y')) ?></p> <p><?= __('AUR packages are user produced content. Any use of the provided files is at your own risk.') ?></p> </div> </div> -- 2.6.2

1 0

[aur-dev] [PATCH 1/2] Fix user name length limit on the account edit form
by Lukas Fleischer 25 Oct '15

25 Oct '15

Change the maxlength attribute of the user name input field such that it corresponds to the username_max_len option. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/template/account_edit_form.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/template/account_edit_form.php b/web/template/account_edit_form.php index 16655c0..b25ff39 100644 --- a/web/template/account_edit_form.php +++ b/web/template/account_edit_form.php @@ -17,7 +17,7 @@ <fieldset> <p> <label for="id_username"><?= __("Username") ?>:</label> - <input type="text" size="30" maxlength="64" name="U" id="id_username" value="<?= htmlspecialchars($U,ENT_QUOTES) ?>" /> (<?= __("required") ?>) + <input type="text" size="30" maxlength="<?= config_get_int('options', 'username_max_len'); ?>" name="U" id="id_username" value="<?= htmlspecialchars($U,ENT_QUOTES) ?>" /> (<?= __("required") ?>) </p> <?php # Only TUs or Devs can promote/demote/suspend a user -- 2.6.2

1 1

[aur-dev] [PATCH] aurblup: Strip version suffixes from provisions
by Lukas Fleischer 24 Oct '15

24 Oct '15

Since we cannot handle versioned provides, simply strip the version suffix and treat them as if they were not versioned. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- scripts/aurblup.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/scripts/aurblup.py b/scripts/aurblup.py index 5397528..325ef3d 100755 --- a/scripts/aurblup.py +++ b/scripts/aurblup.py @@ -4,6 +4,7 @@ import configparser import mysql.connector import os import pyalpm +import re config = configparser.RawConfigParser() config.read(os.path.dirname(os.path.realpath(__file__)) + "/../conf/config") @@ -32,7 +33,9 @@ for sync_db in sync_dbs: blacklist.add(pkg.name) [blacklist.add(x) for x in pkg.replaces] providers.add((pkg.name, pkg.name)) - [providers.add((pkg.name, x)) for x in pkg.provides] + for provision in pkg.provides: + provisionname = re.sub(r'(<|=|>).*', '', provision) + providers.add((pkg.name, provisionname)) db = mysql.connector.connect(host=aur_db_host, user=aur_db_user, passwd=aur_db_pass, db=aur_db_name, -- 2.6.2

1 0

[aur-dev] [PATCH] Highlight broken dependencies
by Lukas Fleischer 24 Oct '15

24 Oct '15

If a dependency neither exists in the official repositories nor in the AUR, make it appear bold red. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/html/css/aurweb.css | 5 +++++ web/lib/pkgfuncs.inc.php | 9 +++++++++ 2 files changed, 14 insertions(+) diff --git a/web/html/css/aurweb.css b/web/html/css/aurweb.css index 4fb256f..11af747 100644 --- a/web/html/css/aurweb.css +++ b/web/html/css/aurweb.css @@ -147,3 +147,8 @@ label.confirmation, #merge-into { width: auto; } + +#pkgdepslist .broken { + color: red; + font-weight: bold; +} diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 3108241..cedc360 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -336,6 +336,15 @@ function pkg_depend_link($name, $type, $cond, $arch, $pkg_id) { */ $providers = pkg_providers($name); + if (count($providers) == 0) { + $link = '<span class="broken">'; + $link .= htmlspecialchars($name); + $link .= '</span>'; + $link .= htmlspecialchars($cond) . ' '; + $link .= pkg_deplink_annotation($type, $arch, $desc); + return $link; + } + $link = htmlspecialchars($name); foreach ($providers as $provider) { if ($provider[1] == $name) { -- 2.6.2

1 0

[aur-dev] [PATCH] Hide optdepend description if not available
by Lukas Fleischer 24 Oct '15

24 Oct '15

Remove the "(unknown)" suffix that used to be shown for optional dependencies without a description. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/lib/pkgfuncs.inc.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index dab4315..3108241 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -324,7 +324,7 @@ function pkg_depend_link($name, $type, $cond, $arch, $pkg_id) { $name = $tokens[0]; $desc = $tokens[1]; } else { - $desc = '(unknown)'; + $desc = ''; } /* -- 2.6.2

1 0

[aur-dev] [PATCH] Include trivial AUR providers in pkg_providers()
by Lukas Fleischer 24 Oct '15

24 Oct '15

When obtaining provisions using pkg_providers(), we already include virtual providers from the official repositories, virtual providers from the AUR and trivial providers (i.e. packages having the given name) from the official repositories. Include trivial providers from the AUR as well. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/lib/pkgfuncs.inc.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 34ec2c5..dab4315 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -164,10 +164,11 @@ function pkg_groups($pkgid) { function pkg_providers($name) { $dbh = DB::connect(); $q = "SELECT p.ID, p.Name FROM Packages p "; - $q.= "INNER JOIN PackageRelations pr ON pr.PackageID = p.ID "; - $q.= "INNER JOIN RelationTypes rt ON rt.ID = pr.RelTypeID "; - $q.= "WHERE rt.Name = 'provides' "; - $q.= "AND pr.RelName = " . $dbh->quote($name); + $q.= "LEFT JOIN PackageRelations pr ON pr.PackageID = p.ID "; + $q.= "LEFT JOIN RelationTypes rt ON rt.ID = pr.RelTypeID "; + $q.= "WHERE p.Name = " . $dbh->quote($name) . " "; + $q.= "OR (rt.Name = 'provides' "; + $q.= "AND pr.RelName = " . $dbh->quote($name) . ")"; $q.= "UNION "; $q.= "SELECT 0, Name FROM OfficialProviders "; $q.= "WHERE Provides = " . $dbh->quote($name); -- 2.6.2

1 0

[aur-dev] [PATCH] Fix spacing of versioned dependencies
by Lukas Fleischer 24 Oct '15

24 Oct '15

Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/lib/pkgfuncs.inc.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/lib/pkgfuncs.inc.php b/web/lib/pkgfuncs.inc.php index 717085d..82408fc 100644 --- a/web/lib/pkgfuncs.inc.php +++ b/web/lib/pkgfuncs.inc.php @@ -347,7 +347,7 @@ function pkg_depend_link($name, $type, $cond, $arch, $pkg_id) { break; } } - $link .= ' ' . htmlspecialchars($cond); + $link .= htmlspecialchars($cond) . ' '; foreach ($providers as $key => $provider) { if ($provider[1] == $name) { -- 2.6.2

1 0

[aur-dev] [PATCH] Disable mass unflagging
by Lukas Fleischer 24 Oct '15

24 Oct '15

This currently does not work. Disable it until we have proper support for flagging multiple packages on the flag page. Fixes FS#46780. Signed-off-by: Lukas Fleischer <lfleischer(a)archlinux.org> --- web/template/pkg_search_results.php | 1 - 1 file changed, 1 deletion(-) diff --git a/web/template/pkg_search_results.php b/web/template/pkg_search_results.php index cce533c..3046c25 100644 --- a/web/template/pkg_search_results.php +++ b/web/template/pkg_search_results.php @@ -109,7 +109,6 @@ if (!$result): ?> <p> <select name="action"> <option><?= __("Actions") ?></option> - <option value="do_Flag"><?= __("Flag Out-of-date") ?></option> <option value="do_UnFlag"><?= __("Unflag Out-of-date") ?></option> <option value="do_Adopt"><?= __("Adopt Packages") ?></option> <option value="do_Disown"><?= __("Disown Packages") ?></option> -- 2.6.2

1 0