On 12/14/2009 06:30 AM, Dan McGee wrote:
On Thu, Dec 3, 2009 at 3:17 AM, Allan McRae <allan@archlinux.org> wrote:
Cedric Staniewski wrote:
On 12/03/2009 05:19 AM, Allan McRae wrote:
Cedric Staniewski wrote:
Signed-off-by: Cedric Staniewski <cedric@gmx.ca> --- doc/makepkg.8.txt | 15 +++++++++++++++ 1 files changed, 15 insertions(+), 0 deletions(-)
diff --git a/doc/makepkg.8.txt b/doc/makepkg.8.txt index 703c1b0..ccb9a28 100644 --- a/doc/makepkg.8.txt +++ b/doc/makepkg.8.txt @@ -170,6 +170,21 @@ Environment Variables must be supported by this command. If the variable is not set or empty, makepkg will fall back to `pacman'. +**PKGDEST=**"/path/to/folder":: + If this value is not set, packages will by default be placed in the + current directory (location of the linkman:PKGBUILD[5]). Many people + like to keep all their packages in one place so this option allows + this behavior. A common location is ``/home/packages''. + This environment variable will override the corresponding value + defined in linkman:makepkg.conf[5].
I do not like the repetition of makepkg.conf.5 here. I'd prefer just **PKGDEST=**"/path/to/folder":: Overrides the corresponding value defined in linkman:makepkg.conf[5].
Or something quite simple like that. People can then look up makepkg.conf.5 to find out what the variable does.
I agree with you. How about
**PKGDEST=**"/path/to/folder":: Folder where the resulting packages will be stored. Overrides the corresponding value defined in linkman:makepkg.conf[5].
**SRCDEST=**"/path/to/folder":: Folder where the downloaded sources will be stored. Overrides the corresponding value defined in linkman:makepkg.conf[5].
Looks good to me. I'll let Dan comment as he always picks holes in my documentation! :P
The hole I will pick is I don't believe this documentation is true. Why would we prefer an environment variable declaration over a sourced PKGDEST= set of the value?
I do not know why support for these environment variables was originally added but the responsible commit is 1def746 [1]. [1] http://projects.archlinux.org/pacman.git/commit/?id=1def746ad5f1024c78db1935...