[40593] trunk/dports/lang/gcj34/Portfile

[Ryan Schmidt]

On Oct 8, 2008, at 01:03, jmr at macports.org wrote:

> Revision: 40593
>           http://trac.macports.org/changeset/40593
> Author:   jmr at macports.org
> Date:     2008-10-07 23:03:12 -0700 (Tue, 07 Oct 2008)
> Log Message:
> -----------
> gcj34: remove use of cd
>
> Modified Paths:
> --------------
>     trunk/dports/lang/gcj34/Portfile
>
> Modified: trunk/dports/lang/gcj34/Portfile
> ===================================================================
> --- trunk/dports/lang/gcj34/Portfile	2008-10-08 03:52:39 UTC (rev  
> 40592)
> +++ trunk/dports/lang/gcj34/Portfile	2008-10-08 06:03:12 UTC (rev  
> 40593)
> @@ -49,25 +49,24 @@
>
>  # Since we install in a subdir dedicated to gcj, this gets it  
> visibility
>  post-destroot {
> -    cd ${destroot}${prefix}/${name}-${version}/share/man/
>      foreach n {1 7} {
>          foreach f [glob man${n}/*.${n}] {

Doesn't this glob now fail? The glob relied on the current directory  
being the man directory to find the man${n} directories. I think you  
want the "-directory" switch to the glob command.

http://tmml.sourceforge.net/doc/tcl/glob.html

> -            system "gzip -9 ${f}"
> +            system "cd ${destroot}${prefix}/${name}-${version}/ 
> share/man/ && gzip -9 ${f}"

And then you would revert this change, since ${f} will then contain  
the full path to the file.

I would test it on my PowerPC machine except I can't access it until  
tomorrow.

>          }
>      }

[snip]