[macports-ports] branch master updated: aacgain: Update dirname in post-extract phase

[Ryan Schmidt]

On Apr 8, 2018, at 04:12, Jackson Isaac wrote:

> Jackson Isaac (JacksonIsaac) pushed a commit to branch master
> in repository macports-ports.
> 
> 
> https://github.com/macports/macports-ports/commit/65defdc294622caa6e4c44d874286917a1d51ce1
> 
> The following commit(s) were added to refs/heads/master by this push:
> 
>      new 65defdc  aacgain: Update dirname in post-extract phase
> 
> 65defdc is described below
> 
> 
> commit 65defdc294622caa6e4c44d874286917a1d51ce1
> 
> Author: ijackson
> AuthorDate: Sun Apr 8 14:42:19 2018 +0530
> 
> 
>     aacgain: Update dirname in post-extract phase
>     
>     Version and tarball name didn't match and
>     hence there was an issue while applying patch.
> 
> ---
>  audio/aacgain/Portfile | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/audio/aacgain/Portfile b/audio/aacgain/Portfile
> index 481b021..bda6c74 100644
> --- a/audio/aacgain/Portfile
> +++ b/audio/aacgain/Portfile
> @@ -21,6 +21,10 @@ use_bzip2           yes
>  checksums           sha256  2bb8e27aa8f8434a4861fdbc70adb9cb4b47e1dfe472910d62d6042cb80a2ee1 \
>                      rmd160  8f97121f3797cde45f71839dbe14251b74365ccd
>  
> +post-extract {
> +    file rename ${workpath}/${name}-1.8 ${workpath}/${name}-${version}
> +}

Set worksrcdir instead of this.

However, don't do that, because this update was completely wrong. No version 1.9 exists at the download site. Take a look:

http://aacgain.altosdesign.com/alvarez/

There is only 1.8. Their web server is just configured in an improperly helpful manner: If you request the file for the nonexistent 1.9, it silently redirects you to the file for 1.8:

$ curl -I http://aacgain.altosdesign.com/alvarez/aacgain-1.9.tar.bz2
HTTP/1.1 301 Moved Permanently
Content-Type: text/html; charset=iso-8859-1
Connection: keep-alive
Keep-Alive: timeout=15
Date: Sun, 08 Apr 2018 09:26:54 GMT
Server: Apache
Location: http://aacgain.altosdesign.com/alvarez/aacgain-1.8.tar.bz2

What I did not see in your previous commit which updated the port to "1.9", because you combined whitespace changes with functional ones, is that the checksums did not change. So what you've called 1.9 here is definitely unchanged from 1.8.

The web site claims a version 1.9 exists, but I don't know where it is. If it can't be found, the changes you made should be reverted to put the port back at version 1.8. The epoch will have to be increased so that anybody who did install 1.9 already will be downgraded back to 1.8.

It seems that you didn't try to build this before committing it, or you would have at least seen the patch failure. Please always verify successful installation of a port on your own system before committing.