Bug 246273 - sysutils/aimage: [MAINTAINER] fixes build on 12 && 13
Summary: sysutils/aimage: [MAINTAINER] fixes build on 12 && 13
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs mailing list
URL:
Keywords: buildisok, easy, patch, patch-ready
Depends on:
Blocks:
 
Reported: 2020-05-07 08:23 UTC by Chris Hutchinson
Modified: 2020-05-12 00:56 UTC (History)
1 user (show)

See Also:
portmaster: maintainer-feedback+


Attachments
svn diff for sysutils/aimage fixes build on 12 and 13 (1.38 KB, patch)
2020-05-07 08:23 UTC, Chris Hutchinson
portmaster: maintainer-approval+
Details | Diff
Build report for sysutils/aimage (13.38 KB, text/plain)
2020-05-07 08:24 UTC, Chris Hutchinson
no flags Details
aimage.diff (2.75 KB, patch)
2020-05-10 18:02 UTC, Tobias Kortkamp
no flags Details | Diff
Revised version of patch provided by tobik@ (2.39 KB, patch)
2020-05-11 21:00 UTC, Chris Hutchinson
portmaster: maintainer-approval+
Details | Diff
QA log for 13/r358902/amd64 (14.07 KB, text/plain)
2020-05-11 21:03 UTC, Chris Hutchinson
no flags Details
QA log on 11.x (13.67 KB, text/plain)
2020-05-12 00:56 UTC, Chris Hutchinson
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Hutchinson 2020-05-07 08:23:47 UTC
Created attachment 214220 [details]
svn diff for sysutils/aimage fixes build on 12 and 13

sysutils/aimage is currently marked broken on 12 and 13
The svn diff attached to this pr(1) corrects that.

CHANGES

Makefile

tested fine on 13/amd64 - see BUILD report (also attached)

That's it.

Thanks!

--Chris
Comment 1 Chris Hutchinson 2020-05-07 08:24:49 UTC
Created attachment 214221 [details]
Build report for sysutils/aimage
Comment 2 daniel.engberg.lists 2020-05-07 08:37:26 UTC
Please use poudriere and the SSL workarounds seems very hackish.
Comment 3 Chris Hutchinson 2020-05-07 08:53:38 UTC
(In reply to daniel.engberg.lists from comment #2)
It's needed because it's less disruptive for those
on 11 which has (Open)SSL in base, and use it. This
tactic is used in many other ports as well --
openntpd, relayd, dsniff, to name only 3.
Comment 4 Chris Hutchinson 2020-05-07 09:04:44 UTC
(In reply to daniel.engberg.lists from comment #2)
If it builds in a fresh install on bare metal. It's
a better litmus test than poudriere.
Comment 5 Automation User 2020-05-10 17:34:57 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/144521461
Comment 6 Tobias Kortkamp freebsd_committer 2020-05-10 18:02:10 UTC
Created attachment 214362 [details]
aimage.diff

Try this instead. It should make it buildable with OpenSSL 1.1.
Comment 7 Chris Hutchinson 2020-05-10 18:34:37 UTC
(In reply to Tobias Kortkamp from comment #6)
I just took a peek. Yes, I can see it will
definitely work. I'll test your patch against
11 and 13 and update this pr accordingly.

Thanks Tobias! Greatly appreciated.

--Chris
Comment 8 Chris Hutchinson 2020-05-11 21:00:00 UTC
Created attachment 214396 [details]
Revised version of patch provided by tobik@

This is a revision of the one provided by tobik@
It simply removes the trailing noise from svn.
The attached patch works as intended on 13/amd64.
I will attach the QA log from that build, as well.

--Chris
Comment 9 Chris Hutchinson 2020-05-11 21:03:35 UTC
Created attachment 214397 [details]
QA log for 13/r358902/amd64

This is a QA log that proves the most recent patch
tests fine on 13/r358902/amd64

Testing on 11/amd64 now.
QA log coming real soon. :-)

--Chris
Comment 10 Chris Hutchinson 2020-05-12 00:56:24 UTC
Created attachment 214400 [details]
QA log on 11.x

OK here's the QA log that confirms the proposed patch
(svn diff) works as intended on 11.x

This should do it. The proposed patch works as intended
on 11.x as well as 13 (amd64).

Please commit.

Thanks Tobias! :-)

--Chris