Regression in PREFIX handling in packages

[Alexander Leidinger]

On Wed, 29 Aug 2012 22:02:46 +0200 Alexander Leidinger
<Alexander at Leidinger.net> wrote:

> And this one is tested (copy&paste, may have lost tabs and add
> linebreaks from my mailer):
> ---snip---
> # svn diff
> Index: perform.c
> ===================================================================
> --- perform.c   (revision 239708)
> +++ perform.c   (working copy)
> @@ -215,10 +215,15 @@
> 
>      /* Prefix should add an @cwd to the packing list */
>      if (Prefix) {
> -        char resolved_prefix[PATH_MAX];
> -        if (realpath(Prefix, resolved_prefix) == NULL)
> -           err(EXIT_FAILURE, "couldn't resolve path for prefix: %s",
>      Prefix);
> -       add_plist_top(&plist, PLIST_CWD, resolved_prefix);
> +       if (Prefix[0] != '/') {
> +               char resolved_prefix[PATH_MAX];
> +               if (realpath(Prefix, resolved_prefix) == NULL)
> +                       err(EXIT_FAILURE,
> +                           "couldn't resolve path for prefix: %s",
>      Prefix);
> +               add_plist_top(&plist, PLIST_CWD, resolved_prefix);
> +       } else {
> +               add_plist_top(&plist, PLIST_CWD, Prefix);
> +       }
>      }
> 
>      /* Add the origin if asked, at the top */
> ---snip---

Can I go ahead and commit this?

Bye,
Alexander.

-- 
http://www.Leidinger.net    Alexander @ Leidinger.net: PGP ID = B0063FE7
http://www.FreeBSD.org       netchild @ FreeBSD.org  : PGP ID = 72077137