Re: [Module::Build] .modulebuildrc documentation out of sync with code
Status: Beta
Brought to you by:
kwilliams
|
From: Randy W. S. <ml...@th...> - 2006-04-06 18:23:12
|
David Golden wrote:
> I posted this to RT, but I'll copy here as well:
>
> Documentation for the location of .modulebuildrc says .modulebuildrc
> should be placed in $ENV{HOME}, but the code actually searches the
> following environment variables: APPDATA HOME USERPROFILE WINDIR SYS$LOGIN.
>
> This is particularly confusing because HOME isn't listed first, so a
> .modulebuildrc file place in $ENV{HOME} just mysteriously fails to work
> on Win32 where $ENV{APPDATA} exists.
Instead of search for the first valid "HOME" directory, we should
probably be looking for the first one that contains the file we need
since all of the directories usually exist.
I'm not sure about the order of the search. I would put "HOME" first
because if it is explicitly set (esp on Windows), then it is probably
intended to be the "primary" home dir.
Having trouble finding the original report though that led to the above
search being added, so I'm not sure of the exact problem it was intended
to solve. I don't want to break anything.
Randy.
|