Bug 242287 - sysutils/ufetch: Update to 0.2
Summary: sysutils/ufetch: Update to 0.2
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: freebsd-ports-bugs (Nobody)
URL: https://gitlab.com/jschx/ufetch/-/tag...
Keywords: buildisok
Depends on:
Blocks:
 
Reported: 2019-11-28 20:44 UTC by Lewis Cook
Modified: 2020-01-01 21:32 UTC (History)
2 users (show)

See Also:


Attachments
Updated port patch (1.40 KB, patch)
2019-11-28 20:44 UTC, Lewis Cook
lcook: maintainer-approval+
Details | Diff
ufetch-0.2.diff (1.47 KB, patch)
2019-12-31 16:11 UTC, Lewis Cook
lcook: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lewis Cook freebsd_committer freebsd_triage 2019-11-28 20:44:16 UTC
Created attachment 209516 [details]
Updated port patch

In the initial port submission, DISTVERSION was incorrectly set (rather, needed to use the date of the commit instead to avoid confusion). This is now remedied in the patch, updated to the latest commit and bumped both PORTREVISION and PORTEPOCH (plus corresponding distinfo). Also made a few minor tweaks to the Makefile for clarity.
Comment 1 Automation User 2019-11-28 20:48:53 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/99330817
Comment 2 Koichiro Iwao freebsd_committer freebsd_triage 2019-12-09 09:15:52 UTC
I think bumping PORTEPOCH is enough but am not really sure.
Comment 3 Lewis Cook freebsd_committer freebsd_triage 2019-12-09 09:28:06 UTC
(In reply to Koichiro Iwao from comment #2)

Thanks for your response. I've been referring to the porters handbook in this particular instance, and more specifically it's noted:

> People using binary packages will never see the update if PORTREVISION is not bumped. Without increasing PORTREVISION, the package builders have no way to detect the change and thus, will not rebuild the package.

As we're changing the DISTVERSION to 'g20191028' when it was previously '0.1-1', (from what I've read) there could be a potential issue where it's not seen as an update, but rather a downgrade in version. Which lead to me bumping both of the values avoiding any conflict―forcing a rebuild of the package and prompting users of a new update.

Although I'm not 100% certain if this is completely necessary. If a change is needed I'm more than happy to do so. Cheers!
Comment 4 Lewis Cook freebsd_committer freebsd_triage 2019-12-31 16:09:51 UTC
Awesome, a new tag release (v0.2) was pushed -- meaning everything I said above can be discarded. Submitting new patch now.
Comment 5 Lewis Cook freebsd_committer freebsd_triage 2019-12-31 16:11:35 UTC
Created attachment 210357 [details]
ufetch-0.2.diff

* Added DISTVERSIONPREFIX and bumped DISTVERSION;
* Updated distinfo.
Comment 6 Tobias C. Berner freebsd_committer freebsd_triage 2020-01-01 21:32:05 UTC
Committed. Thanks.
Comment 7 commit-hook freebsd_committer freebsd_triage 2020-01-01 21:32:36 UTC
A commit references this bug:

Author: tcberner
Date: Wed Jan  1 21:31:38 UTC 2020
New revision: 521777
URL: https://svnweb.freebsd.org/changeset/ports/521777

Log:
  sysutils/ufetch: Update to 0.2

  PR:		242287
  Submitted by:	Lewis Cook <vulcan@wired.sh> (maintainer)

Changes:
  head/sysutils/ufetch/Makefile
  head/sysutils/ufetch/distinfo