Bug 211110 - devel/vasm: Fix MASTER_SITES, Remove broken test syntax, Update to 1.7e
Summary: devel/vasm: Fix MASTER_SITES, Remove broken test syntax, Update to 1.7e
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: Thomas Zander
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-07-14 12:34 UTC by Carsten Larsen
Modified: 2016-09-18 12:09 UTC (History)
3 users (show)

See Also:
riggs: merge-quarterly+


Attachments
Suggested update (2.40 KB, patch)
2016-07-14 12:34 UTC, Carsten Larsen
no flags Details | Diff
Diff to only change mastersite (1.18 KB, patch)
2016-07-14 13:13 UTC, Carsten Larsen
no flags Details | Diff
vasm-1.7e.diff (2.41 KB, patch)
2016-09-10 20:39 UTC, Carsten Larsen
cs: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carsten Larsen 2016-07-14 12:34:30 UTC
Created attachment 172506 [details]
Suggested update

The mastersite is pointing to a location which is used for rolling releases. Current distfile is fetched from distcache instead.

Suggested changes are attached.
* Changed mastersite to avoid fetching rolling releases
* devel/vasm: Update to 1.7e
* Removed broken test syntax
Comment 1 Kubilay Kocak freebsd_committer freebsd_triage 2016-07-14 12:40:17 UTC
It would be great to separate fixes from updates, so that the former can be MFH'd to the quarterly branch.

If you can provide two separate patches, please do so

If not, can the committer who assigns this issue to themselves please commit the fixes from the update separately
Comment 2 Carsten Larsen 2016-07-14 13:13:41 UTC
Created attachment 172510 [details]
Diff to only change mastersite

According to the website "http://sun.hasenbraten.de/vasm/daily/vasm.tar.gz" is a "daily snapshot of the full vasm source from the CVS". Nothing bad happened.

portlint: OK (looks fine.)
testport: OK (poudriere: amd64, 10.3 tested)
Comment 3 Uffe Jakobsen 2016-09-04 20:27:24 UTC
Hi,

Apologies for the late reply

I'm in a situation that leaves little time to maintain this package.

I've communicated with the reporter Carsten Larsen and he is willing to adopt this package - I do not know if this is the official way of passing ownership ? but it is ok with me
Comment 4 Carsten Larsen 2016-09-10 20:39:57 UTC
Created attachment 174630 [details]
vasm-1.7e.diff

Take maintainership and update to 1.7e.
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-09-18 11:43:12 UTC
A commit references this bug:

Author: riggs
Date: Sun Sep 18 11:42:36 UTC 2016
New revision: 422374
URL: https://svnweb.freebsd.org/changeset/ports/422374

Log:
  Update to 1.7e, fix MASTER_SITES, new maintainer

  Detailed log:
  - Update to upstream version 1.7e
  - Changed MASTER_SITES, to avoid fetching rolling releases
  - Removed broken test syntax
  - New maintainer: cs@innolan.dk

  PR:		211110
  Submitted by:	cs@innolan.dk (new maintainer)
  Approved by:	uffe@uffe.org (previous maintainer)
  MFH:		2016Q3 (build fix blanket)

Changes:
  head/devel/vasm/Makefile
  head/devel/vasm/distinfo
  head/devel/vasm/pkg-plist
Comment 6 commit-hook freebsd_committer freebsd_triage 2016-09-18 11:55:14 UTC
A commit references this bug:

Author: riggs
Date: Sun Sep 18 11:54:58 UTC 2016
New revision: 422379
URL: https://svnweb.freebsd.org/changeset/ports/422379

Log:
  MFH: r422374 (only the MASTER_SITES fix from r422374)

  Detailed log:
  - Changed MASTER_SITES, to avoid fetching rolling releases

  PR:		211110
  Submitted by:	cs@innolan.dk (new maintainer)
  Approved by:	ports-secteam (build fix blanket)

Changes:
_U  branches/2016Q3/
  branches/2016Q3/devel/vasm/Makefile
  branches/2016Q3/devel/vasm/distinfo
Comment 7 Thomas Zander freebsd_committer freebsd_triage 2016-09-18 12:09:24 UTC
Thank you both Uffe and Carsten for taking care of this port previously and from now on. It is very much appreciated!