Bug 206848 - ports-mgmt/poudriere-devel: jail fetching breaks with new download.freebsd.org path, requires FREEBSD_HOST modification
Summary: ports-mgmt/poudriere-devel: jail fetching breaks with new download.freebsd.or...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Bryan Drewery
URL:
Keywords: easy, needs-patch, patch
Depends on:
Blocks:
 
Reported: 2016-02-02 13:58 UTC by p5B2EA84B3
Modified: 2016-02-16 00:15 UTC (History)
3 users (show)

See Also:
koobs: maintainer-feedback? (bdrewery)
koobs: merge-quarterly?

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description p5B2EA84B3 2016-02-02 13:58:22 UTC 
Solution to /usr/local/share/poudriere/jail.sh:
534c534
< 			URL="${FREEBSD_HOST}/ftp/${type}/${ARCH}/${V}" ;;
---
> 			URL="${FREEBSD_HOST}/pub/FreeBSD/${type}/${ARCH}/${V}" ;;
587c587
< 				URL="${FREEBSD_HOST}/ftp/${type}/${ARCH%%.*}/${ARCH##*.}/${V}"
---
> 				URL="${FREEBSD_HOST}/pub/FreeBSD/${type}/${ARCH%%.*}/${ARCH##*.}/${V}"
Comment 1 p5B2EA84B3 2016-02-02 14:01:20 UTC 
Forgot to mention Poudriere version: 3.2-pre
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2016-02-02 14:11:18 UTC 
Could you please:

* Mention which port specifically: ports-mgmt/ poudriere or poudriere-devel?
* Provide the proposed change as a unified diff, and as an attachment 

Thank you for your report
Comment 3 Tobias Kortkamp freebsd_committer freebsd_triage 2016-02-03 08:05:28 UTC 
The summary change is not correct. This also happens with -m http.

I was able to solve this by changing FREEBSD_HOST in /usr/local/etc/poudriere.conf to https://download.freebsd.org.

I am using ports-mgmt/poudriere-devel.
Comment 4 Tobias Kortkamp freebsd_committer freebsd_triage 2016-02-03 08:12:31 UTC 
I see that using download.freebsd.org is also suggested by /usr/local/etc/poudriere.conf.sample now.
Comment 5 Bryan Drewery freebsd_committer freebsd_triage 2016-02-03 20:18:52 UTC 
This is an upgrade/POLA issue I'll have to work out.
Comment 6 p5B2EA84B3 2016-02-07 16:12:57 UTC 
Version here is poudriere-devel-3.1.99.20160127
Comment 8 commit-hook freebsd_committer freebsd_triage 2016-02-16 00:15:33 UTC 
A commit references this bug:

Author: bdrewery
Date: Tue Feb 16 00:14:39 UTC 2016
New revision: 408965
URL: https://svnweb.freebsd.org/changeset/ports/408965

Log:
  - Update to 3.0-1639-ga5325bf

  Changes:
    - Optimize jail startup/cloning/rollback and mtree comparisons.
    - api: Make more useful
    - check_fs_violation: Use mktemp rather than hard-coded tmpfile
    - Fix false-positive 'Leftover processes' warning after disabling jexecd
    - Conditionalize the jexec logic around USE_JEXECD and add back missing jkills.
    - QEMU: Remove _MAKE_JOBS not present in release-3.1 and noted to break
      builds by sbruno
    - Kill processes before remounting/rolling back the builder FS, rather than
      after.
    - jail -c: Support older FREEBSD_HOST=ftp.freebsd.org URL schemes still. [1]
    - jail -c: Support pre-distributed MANIFESTs for checksum comparing.

  PR:		206848 [1]

Changes:
  head/ports-mgmt/poudriere-devel/Makefile
  head/ports-mgmt/poudriere-devel/distinfo