- Aur-dev - lists.archlinux.org

[aur-dev] AUR Coding recomendations IMPORTANT
by Angel Velásquez 21 Jun '08

21 Jun '08

Hi, I am writing this e-mail after reviewing the AUR code and some patches that people have been sending. As we know, AUR (like many applications) is still in development, and many of the things that are actually working weren't coded by any of us, and I can understand this. The purpose of my e-mail is to give some recommendations that should be adopted by everyone, I don't want to be arrogant or force you to do something without knowing why, so I will explain the reasons behind my recommendations and then you will probably agree with me. Singles quotes vs Doubles quotes ===================== Problem: In the past I sent an e-mail about this topic, but I think I was misunderstood. The correct way should be using singles quotes when you don't need to parse any kind of variable, that applies in any case. Examples: 1.- include "lib/whatever.inc.php"; // why double quotes ? just to waste resources and no more.. 2.- function myFunction($var) { return "Hello ".$var; // wrong concatenation AND using double quotes ?! # return 'Hello '.$var; // this is the correct use } echo myFunction("world"); // wrong # echo myFunction('world'); // Oh nice you know what i'm talking about. 3.- $_SESSION["myName"] = "MyNameIsWrong"; // and it certainly is wrong, thanks for wasting resources with double quotes! # $_SESSION['myName'] = 'Foo Bar'; // that's the way 4.- // the last example of a block of HTML print "<table>\n"; print "<tr>\n"; print "<td>".$title."</td>\n"; print "</tr>\n"; print "</table>\n"; // Come on!!!!, the following line does the same. echo "<table>\n<tr>\n<td>$title</td>\n</tr>\n</table>"; // Yes I know that placing plenty of \n's would render the code unreadable, but you can do it in one line with a simple echo. Overview: I recommend the use of double quotes in the case that you need to parse many variables so that the code doesn't become unreadable, or if you want to output HTML code in a sorted way. print vs echo ======== Problem: Many of us usually prefer to use print because we are used to the function name similarities from languages other than PHP, and we can't get used to *not* using it. Examples: 1.- print 'echo is ugly!'; // assuming that you've learned about singles and doubles quotes :D Overview: Print outputs the string and has an actual return value of TRUE or FALSE, so that converts print in a function slower than echo. See this thread for more information: http://codingforums.com/archive/index.php?t-41745.html To wrap this up, I just want to emphasize the fact that whereas this might look trivial and nearly irrelevant to some of you, it is important in order to make AUR a better tool. Cheers. -- Angel Velásquez angvp @ irc.freenode.net Linux Counter: #359909 Arch Linux Trusted User

4 4

[aur-dev] [PATCH] Use bash script to parse pkgbuilds
by Callan Barrett 20 Jun '08

20 Jun '08

Ok this is a test to get a bash script to parse pkgbuilds rather than the current crazy way it's done now. The pkgbuild is sent through a bash script (thanks Xilon) and the vars are output so the result can be evaluated by PHP, this way nothing can get missed and there are no bugs with what's parsed. This also gets rid of the package contents stuff, didn't really have a choice but it seems it's not used for anything, only wasted space. As far as I can tell this fixes about 4 bugs currently open on the bug tracker so aside from being a huge improvement over the current parsing stuff (has anyone ever looked at what that parses, wow) it also fixes a lot of bugs. The only downside to it is we can't tell whether a variable actually exists or is just empty so no more hand holding and doing some faux validation, we parse what we need and the rest of the pkgbuild is up to the maintainer. Although I'm not suggesting we apply *this* patch to the AUR I am suggesting we apply some cleaned up form of it since it would be a great improvement. This patch does work fine by the way it's just a bit messy. -- Callan Barrett

5 10

[aur-dev] [PATCH] Remove all vim mode lines. Add HACKING file.
by Loui 19 Jun '08

19 Jun '08

I made this patch after a little discussion on IRC. Please comment. Cheers!

3 4

[aur-dev] [PATCH] Reorganise web/README.txt and elaborate on PEAR and File_Find.
by Loui 17 Jun '08

17 Jun '08

I'll go ahead and push this, since it's a minor thing.

1 1

[aur-dev] [PATCH] Clean up of pkgsubmit.php
by Callan Barrett 15 Jun '08

15 Jun '08

Just general cleaning up of pkgsubmit.php -- Callan Barrett

1 1

[aur-dev] [PATCH] Updated Italian translation
by Giovanni Scafora 12 Jun '08

12 Jun '08

Hello guys, here is the updated Italian translation. Please, push it to testing. Thanks a lot. -- Giovanni Scafora Arch Linux Trusted User (voidnull) AUR & Pacman Italian Translations http://www.archlinux.it

2 1

[aur-dev] [PATCH] Use PEAR modules when submitting PKGBUILDs
by Callan Barrett 11 Jun '08

11 Jun '08

pkgsubmit.php now uses the PEAR modules Archive_Tar and File_Find for finding the PKGBUILD and messing with tarballs, Archive_Tar should be installed by default but you'll need to install File_Find yourself to make this work. I've also done some other cleanup such as (aside from rm_rf) no more exec use and changing some variables around. As far as I can tell this setup is actually faster than what we have now and I know it's much cleaner (compared to before, that file is still a mess) and it's also much more reliable for finding files. Fixes FS#2900. -- Callan Barrett

2 4

[aur-dev] [PATCH] Split HTML in index.php
by Callan Barrett 11 Jun '08

11 Jun '08

Ok here's another version of that patch to clean up index.php. I think in its current form it's good enough to apply but I'd like to hear anyone else's opinion on it. HTML still needs a cleanup. -- Callan Barrett

1 1

[aur-dev] [PATCH] Fix FS#10016
by Callan Barrett 10 Jun '08

10 Jun '08

Fix entry of URLPath and FSPath to be actual locations, fixes RPC results Gonna have to resubmit packages before results are correct. -- Callan Barrett

2 3

[aur-dev] [PATCH] Add version, username and description to daily new package email.
by Loui 09 Jun '08

09 Jun '08

Someone with the nick 'dmartins' popped in IRC and had a little patch on pastebin for the version and description. I thought the script itself needed some clean up so here it is. I'm using MaintainerUID here instead of SubmitterUID for the username.

2 3