- Aur-dev - lists.archlinux.org

[aur-dev] [PATCH 1/3] scripts/cleanup: use native PHP only
by Dan McGee 09 Aug '11

09 Aug '11

No need to shell out to the system here. Also fix the script so it actually works. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- scripts/cleanup | 20 +++++++++++--------- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/scripts/cleanup b/scripts/cleanup index 4fc9ea2..f287350 100755 --- a/scripts/cleanup +++ b/scripts/cleanup @@ -16,21 +16,23 @@ if (empty($dir)) { } set_include_path(get_include_path() . PATH_SEPARATOR . "$dir/lib"); -include("config.inc"); -include("aur.inc"); -include("pkgfuncs.inc"); - -exec('ls ' . INCOMING_DIR, $files); +include("config.inc.php"); +include("aur.inc.php"); +include("pkgfuncs.inc.php"); $count = 0; +$files = scandir(INCOMING_DIR); foreach ($files as $pkgname) { - if (!package_exists($pkgname)) { - echo 'Removing ' . INCOMING_DIR . "$pkgname\n"; - system('rm -r ' . INCOMING_DIR . $pkgname); + if ($pkgname == '.' || $pkgname == '..') { + continue; + } + $fullpath = INCOMING_DIR . $pkgname; + if (!package_exists($pkgname) && is_dir($fullpath)) { + echo 'Removing ' . $fullpath . "\n"; + rm_tree($fullpath); $count++; } } echo "\nRemoved $count directories.\n"; - -- 1.7.6

3 10

[aur-dev] Doubt in a TODO
by Tiago Sousa 01 Jul '11

01 Jul '11

Hello. I am working on the TODO on aur/support/scripts/newpackage-notify. my ideia is to do a dictionary to save all of the languages and we will know the language in use looking into /etc/locale.gen . i already have a script to do that. My doubt is could the message be saved into the dictionary using tiple quote string? Tiago Sousa

2 2

[aur-dev] [PATCH] Wrap comments in <pre> tags, fixes FS#14391
by Manuel Tortosa 01 Jul '11

01 Jul '11

Exactly this, wrap the comments in <pre> tags allowing patches or at least better build output formatting. Greez

4 4

[aur-dev] [PATCH] Convert the language selection bar in a select form
by Manuel Tortosa 29 Jun '11

29 Jun '11

Subject says all. Greez. Manuel

2 3

[aur-dev] [PATCH] Fix empty depends database insert
by Manuel Tortosa 29 Jun '11

29 Jun '11

In pkgsubmit.php in this part: foreach ($depends as $dep) { $deppkgname = preg_replace("/(<|<=|=|>=|>).*/", "", $dep); $depcondition = str_replace($deppkgname, "", $dep); if ($deppkgname == "#") { break; } $q = sprintf("INSERT INTO PackageDepends (PackageID, DepName, DepCondition) VALUES (%d, '%s', '%s')", $packageID, mysql_real_escape_string($deppkgname), mysql_real_escape_string($depcondition)); db_query($q, $dbh); } This will pass through the loop at least once in case of pkgdesc=(), adding an empty pkgdepends entry to the database. Easy to avoid by the fact $depends = explode(" ", $new_pkgbuild['depends']); returns false. Best Regards.

2 1

[aur-dev] [PATCH 1/3] aurblup: blacklist processing query changes
by Dan McGee 28 Jun '11

28 Jun '11

* Do all list building and freeing outside of the transaction to keep it as short as possible. * Remove ability to blacklist without transactions as we now only support InnoDB/transactional engines with proper relations. * No need to turn autocommit off; BEGIN TRANSACTION operates regardless of this setting. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- scripts/aurblup/aurblup.c | 23 +++++------------------ scripts/aurblup/config.h.proto | 3 --- 2 files changed, 5 insertions(+), 21 deletions(-) diff --git a/scripts/aurblup/aurblup.c b/scripts/aurblup/aurblup.c index fea7ffc..1f7aefc 100644 --- a/scripts/aurblup/aurblup.c +++ b/scripts/aurblup/aurblup.c @@ -122,19 +122,11 @@ blacklist_sync(alpm_list_t *pkgs_cur, alpm_list_t *pkgs_new) { alpm_list_t *pkgs_add, *pkgs_rem, *p; -#if MYSQL_USE_TRANSACTIONS - if (mysql_autocommit(c, 0)) - mysql_die("failed to turn MySQL autocommit off: %s\n"); + pkgs_add = alpm_list_diff(pkgs_new, pkgs_cur, (alpm_list_fn_cmp)strcmp); + pkgs_rem = alpm_list_diff(pkgs_cur, pkgs_new, (alpm_list_fn_cmp)strcmp); if (mysql_query(c, "START TRANSACTION;")) mysql_die("failed to start MySQL transaction: %s\n"); -#else - if (mysql_query(c, "LOCK TABLES PackageBlacklist WRITE;")) - mysql_die("failed to lock MySQL table: %s\n"); -#endif - - pkgs_add = alpm_list_diff(pkgs_new, pkgs_cur, (alpm_list_fn_cmp)strcmp); - pkgs_rem = alpm_list_diff(pkgs_cur, pkgs_new, (alpm_list_fn_cmp)strcmp); for (p = pkgs_add; p; p = alpm_list_next(p)) blacklist_add(alpm_list_getdata(p)); @@ -142,16 +134,11 @@ blacklist_sync(alpm_list_t *pkgs_cur, alpm_list_t *pkgs_new) for (p = pkgs_rem; p; p = alpm_list_next(p)) blacklist_remove(alpm_list_getdata(p)); - alpm_list_free(pkgs_add); - alpm_list_free(pkgs_rem); - -#if MYSQL_USE_TRANSACTIONS if (mysql_query(c, "COMMIT;")) mysql_die("failed to commit MySQL transaction: %s\n"); -#else - if (mysql_query(c, "UNLOCK TABLES;")) - mysql_die("failed to unlock MySQL tables: %s\n"); -#endif + + alpm_list_free(pkgs_add); + alpm_list_free(pkgs_rem); } alpm_list_t * diff --git a/scripts/aurblup/config.h.proto b/scripts/aurblup/config.h.proto index fc1a87b..daf1744 100644 --- a/scripts/aurblup/config.h.proto +++ b/scripts/aurblup/config.h.proto @@ -6,9 +6,6 @@ #define CONFIG_KEY_PASSWD "AUR_db_pass" #define CONFIG_KEY_DB "AUR_db_name" -/* unset this to use "LOCK TABLE" instead of transactions */ -#define MYSQL_USE_TRANSACTIONS 1 - /* libalpm options */ #define ALPM_DBPATH "/var/lib/aurblup/" #define ALPM_MIRROR "ftp://mirrors.kernel.org/archlinux/%s/os/i686" -- 1.7.5.2

2 4

[aur-dev] Python 3 AUR module
by Xyne 27 Jun '11

27 Jun '11

Hi all, I've written a Python 3 AUR module that some of you might find interesting or useful: http://xyne.archlinux.ca/projects/python-aur/ Regards, Xyne

1 0

[aur-dev] misc patches
by Florian Pritz 25 Jun '11

25 Jun '11

I've tested some basic AUR functionality and it still worked fine, so please merge. Cgit (with pull URL) is available at http://git.server-speed.net/users/flo/aur/?h=working

4 10

[aur-dev] epoch fields for packages are completely ignored
by Slavi Pantaleev 24 Jun '11

24 Jun '11

Hey. The epoch field in PKGBUILD files is completely ignored now, and the final Version field for the package consists only of pkgver and pkgrel (example: 5.0-1) This means that various AUR helpers cannot perform updates on packages that have epoch > 0, because the local package is from that higher epoch and the one in the AUR (as reported by rpc.php) is from epoch 0. I'm providing a patch that supposedly fixes the problem by inspecting the epoch field, and if not 0 prepending it to the Version (example: 1:5.0-1). Comments welcome.

2 2

[aur-dev] [PATCH 1/2] Fix performance issues with new PackageDepends lookups
by Dan McGee 24 Jun '11

24 Jun '11

We do a lookup by DepName in the package details view, but I made the silly mistake of forgetting this index addition in the upgrade steps. Signed-off-by: Dan McGee <dan(a)archlinux.org> --- Not sure if this fell through the cracks way back in April... UPGRADING | 6 ++++-- support/schema/aur-schema.sql | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/UPGRADING b/UPGRADING index ad7ad96..51afad3 100644 --- a/UPGRADING +++ b/UPGRADING @@ -9,9 +9,11 @@ each AUR upgrade by running `make install` in the "po/" directory. 2. Remove the "NewPkgNotify" column from the "Users" table: ----- ALTER TABLE Users DROP COLUMN NewPkgNotify; ----- + +3. Fix up issues with depends performance on large dataset. + +ALTER TABLE PackageDepends ADD INDEX (DepName); 3. Rename "web/lib/config.inc" to "web/lib/config.inc.php". diff --git a/support/schema/aur-schema.sql b/support/schema/aur-schema.sql index e64ae82..88d074e 100644 --- a/support/schema/aur-schema.sql +++ b/support/schema/aur-schema.sql @@ -123,6 +123,7 @@ CREATE TABLE PackageDepends ( DepName VARCHAR(64) NOT NULL, DepCondition VARCHAR(20), INDEX (PackageID), + INDEX (DepName), FOREIGN KEY (PackageID) REFERENCES Packages(ID) ON DELETE CASCADE ) ENGINE = InnoDB; -- 1.7.5.4

2 3