Bug 226885
| Summary: | net/ucarp: [PATCH] Switch to Github source tree | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Artyom Davidov <ard_1> | ||||||
| Component: | Individual Port(s) | Assignee: | Yuri Victorovich <yuri> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Some People | CC: | alexey, yuri | ||||||
| Priority: | --- | ||||||||
| Version: | Latest | ||||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| Attachments: |
|
||||||||
Instead of GH_TAGNAME you should use the identifier that git returns: "git describe --tags" Also "touch" shouldn't be needed. Hi Yuri, We cannot use the identifier from "git describe --tags" since there are no tags in that project on GH. So in this particular case we have to fallback to commit identifier. Also the "git describe --always" returns "4f339d4" that is being used in this patch. Regarding those two "touch" statements in the Makefile - Autotools fails to process the source code when those two files are missing. If someone knows a better way to work-around this issue, without adding additional build dependencies to port, please share it with me and I'll update this patch accordingly. wbr, Artyom Loking at the other ports and handbook I wonder woudn't it be better to include the GH commit date in PORTVERSION instead of bumping PORTREVISION in this case? The problem is that the new upstream repository that you change to doesn't indicate that it corresponds to this version. The best solution would have been if they assigned the version, ex. 1.5.3. The next best possible solution is to do what you said: to add date to the version. Though new repository doesn't contain any tags o version identifiers, there is a version number that is present in the source code itself. One can check the configure.ac file and several others and see that they contains 1.5.2 in version fields. Also, this version number can be seen in "ucarp -h" output, after software is build. Considering this I assume that we can set PORTVERSION= 1.5.2.20171201 where 20171201 - is the date of the GH commit with the identifier "4f339d4" that is being referenced in this patch. If these changes are acceptable I'll update this patch accordingly. Created attachment 191874 [details]
patch to use code from GH v2
Added the 2nd version of the patch.
Use PORTVERSION instead of PORTREVISION.
No other changes from the previous version of the patch.
Alexey, Yuri any update on this issue? Maintainer reset. Committed with changes. Thank you for your submission! A commit references this bug: Author: yuri Date: Wed Jul 18 22:41:24 UTC 2018 New revision: 474908 URL: https://svnweb.freebsd.org/changeset/ports/474908 Log: net/ucarp: Switch to Github source tree Port changes: * Add license * Group USEx section * Remove CONFIGURE_ARGS * Use option helpers instead of .if/.else PR: 226885 Submitted by: Artyom Davidov <ard_1@mail.ru> Changes: head/net/ucarp/Makefile head/net/ucarp/distinfo head/net/ucarp/files/patch-l2vlan |
Created attachment 191764 [details] patch to use code from GH Switch to Github source tree since it contains various bugfixes. The original 1.5.2 was released in early 2010 and since then there were no updates to distfiles. But there were numerous bugfixes and minor feature enhancements that are available on Github. Also this patch will do the following things: Fix patch-l2vlan to comply with updated sources. Require GNU Autotools for build. Add a dirty fix in the 'pre-configure' section of the Makefile for two missing 'gettextize' files in upstream code. Don't bump PORTVERSION, just PORTREVISION because upstream is still 1.5.2