Honestly, I'm not convinced $ENV{HOME} should be used on any platform, other than maybe for test purposes. That said, the method of finding specific directory types on Darwin is awfully convoluted - I'm wondering what changing $ENV{HOME} would do when you're looking for a specific type of directory using Mac::Files::FindFolder...

Assuming that you apply the same tests with and without $ENV{HOME} set - including nonsensical values, and that said tests are submitted with your patch, I don't see why Adam would say no. Mind you, I've got a larger patch sitting in his queue at the moment from the beginning of November that hasn't made it in, so I'm not sure what RL is doing to him at the moment.


In reply to Re: Should File::HomeDir honor $ENV{HOME} on Darwin? by Tanktalus
in thread Should File::HomeDir honor $ENV{HOME} on Darwin? by xdg

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":



  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.