Bug 59013 - audio/ripit: Add http proxy support
Summary: audio/ripit: Add http proxy support
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Pav Lucistnik
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-06 16:00 UTC by Simon Barner
Modified: 2003-11-17 23:46 UTC (History)
1 user (show)

See Also:


Attachments
file.diff (2.37 KB, patch)
2003-11-06 16:00 UTC, Simon Barner
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Barner 2003-11-06 16:00:36 UTC
This makes ripit use a http proxy if the HTTP_PROXY environment variable
is set.

Please test this also with non-proxy http und plain cddb mode - I cannot
do that since I am behind a rigid firewall (hence the patch :-)

I have also mailed the (tiny) diff to the author of the script.

How-To-Repeat: 
N/A
Comment 1 Pav Lucistnik freebsd_committer 2003-11-17 19:30:17 UTC
Dear FreeBSD port maintainer,

please take a look at 

http://www.freebsd.org/cgi/query-pr.cgi?pr=59013

Do you approve this change?

-- 
Pav Lucistnik <pav@FreeBSD.org>
              <pav@oook.cz>
Comment 2 Pav Lucistnik freebsd_committer 2003-11-17 19:40:33 UTC
State Changed
From-To: open->feedback

I'll take care of this. 


Comment 3 Pav Lucistnik freebsd_committer 2003-11-17 19:40:33 UTC
Responsible Changed
From-To: freebsd-ports-bugs->pav

I'll take care of this.
Comment 4 Oyvind Moll 2003-11-17 22:19:23 UTC
* pav@FreeBSD.org:
| 
| Dear FreeBSD port maintainer,
| 
| please take a look at 
| 
| http://www.freebsd.org/cgi/query-pr.cgi?pr=59013
| 
| Do you approve this change?

The HTTP proxy change seems fine.

However, the patch in the PR also includes a change of perl
interpreter path from /usr/bin/perl to /usr/local/bin/perl.  This
change should _not_ be applied.  The correct perl path is set through
the use of USE_PERL5 and the post-patch target in the port Makefile.

-- 
   Øyvind Møll
   oyvindmo@initio.no
Comment 5 Simon Barner 2003-11-17 23:21:46 UTC
> The HTTP proxy change seems fine.
> 
> However, the patch in the PR also includes a change of perl
> interpreter path from /usr/bin/perl to /usr/local/bin/perl.  This
> change should _not_ be applied.  The correct perl path is set through
> the use of USE_PERL5 and the post-patch target in the port Makefile.

Oops, sorry. I use the perl5.8 port, and I modified the installed
version of the script to produce my patch. Unfortunately, I did not
review the patch file, this is why this hunk was also submitted.

Simon
Comment 6 Pav Lucistnik freebsd_committer 2003-11-17 23:45:53 UTC
State Changed
From-To: feedback->closed

Committed without shebang change, but with a portrevision bump. 
Thanks!