Bug 214206

Summary: Couldn't fetch distfiles of devel/noweb
Product: Ports & Packages Reporter: Hiroyuki Une <une>
Component: Individual Port(s)Assignee: Dmitry Marakasov <amdmi3>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
error messages when I tried "make fetch" on devel/noweb none

Description Hiroyuki Une 2016-11-04 11:17:42 UTC
Created attachment 176614 [details]
error messages when I tried "make fetch" on devel/noweb

The master site of devel/noweb (ftp://www.eecs.harvard.edu) employs extended passive mode.  Therefore we have to use command EPRT instead of PORT.  However, fetch(1) doesn't support extended passive mode and failed to connect to the master site. 

ftp(1) supports extended passive mode (and has the commands "EPSV" and "EPRT").  So I tried add the following two lines into Makefile and fetch distfiles.  

FETCH_CMD=      /usr/bin/ftp
FETCH_ARGS=

However, I got errors described in the attached file "noweb_errors.txt" because ports(7) appends the option "-S" which is not available for ftp(1).  

I'm afraid that I don't have any patch to resolve this issue.  I think there are two options as follows:

1. append functions for extended passive mode into libfetch, 
2. revise ports/Mk/Scripts/do-fetch.sh to avoid option "-S" 
   when FETCH_CMD=="/usr/bin/ftp".

Unfortunately, I don't have enough skill to do those.  I hope that this article would be helpful.
Comment 1 Dmitry Marakasov freebsd_committer freebsd_triage 2016-12-29 13:03:36 UTC
DISABLE_SIZE does that for you. If you use this port, it would be nice of you to contact the upstream and ask them to provide working (preferrably, plain http) mirror.
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-12-29 13:03:50 UTC
A commit references this bug:

Author: amdmi3
Date: Thu Dec 29 13:03:32 UTC 2016
New revision: 429905
URL: https://svnweb.freebsd.org/changeset/ports/429905

Log:
  - Fix fetch: the only ftp MASTER_SITE requires extended passive mode support from fetcher, so use ftp(1) to fetch

  PR:		214206
  Submitted by:	une@edu.hkg.ac.jp

Changes:
  head/devel/noweb/Makefile