Page MenuHomeFreeBSD

devel/elfio: Updated from 1.0.3 to 3.3
ClosedPublic

Authored by james.wright_digital-chaos.com on Apr 10 2019, 8:29 PM.
Referenced Files
Unknown Object (File)
Thu, Mar 21, 10:26 PM
Unknown Object (File)
Feb 15 2024, 11:16 AM
Unknown Object (File)
Feb 12 2024, 5:01 PM
Unknown Object (File)
Jan 16 2024, 2:29 AM
Unknown Object (File)
Jan 14 2024, 1:11 PM
Unknown Object (File)
Dec 20 2023, 6:05 AM
Unknown Object (File)
Dec 11 2023, 11:55 AM
Unknown Object (File)
Dec 1 2023, 11:06 AM
Subscribers

Details

Summary

Updated ELFIO from 1.0.3 to 3.3. It is not backwards compatible, but no other ports seem to use it.

Bugzilla Ticket:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=237224

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Added elfio3 directory to devel/Makefile
Added myself as maintainer

Added a new port for ELFIO v3.x as it is incompatible with the existing v1.x.x

devel/elfio is unmaintained and no ports depend on it. Why not update it to 3.x instead of adding a new port? Is there a reason devel/elfio needs to stay at 1.x?

devel/elfio3/Makefile
10 ↗(On Diff #56182)

Bad COMMENT (drop "ELFIO is a"). See 5.6. COMMENT.

tobik requested changes to this revision.Apr 14 2019, 6:15 AM
This revision now requires changes to proceed.Apr 14 2019, 6:15 AM

Added a new port for ELFIO v3.x as it is incompatible with the existing v1.x.x

devel/elfio is unmaintained and no ports depend on it. Why not update it to 3.x instead of adding a new port? Is there a reason devel/elfio needs to stay at 1.x?

Wasn't sure if that was the recommended approach or not, so leaned on the side of caution. I'll update devel/efio instead.

james.wright_digital-chaos.com retitled this revision from Added port for devel/elfio3 to devel/elfio: Updated from 1.0.3 to 3.3.
james.wright_digital-chaos.com edited the summary of this revision. (Show Details)

Updated existing devel/elfio port rather than creating a new one (devel/elfio3).

devel/elfio/Makefile
1

^ you may (unfortunately) not remove that line unless you have permission by 'ijliao'

tobik requested changes to this revision.Apr 14 2019, 6:45 PM
tobik added inline comments.
devel/elfio/Makefile
25

Can you install the example source code too like before? IMHO it's more useful than compiled example programs.

The example programs are already installed by the build itself just in the wrong directory, so something is not quite right yet.

$ make check-plist
====> Checking for pkg-plist issues (check-plist)
===> Parsing plist
===> Checking for items in STAGEDIR missing from pkg-plist
Error: Orphaned: bin/anonymizer
Error: Orphaned: bin/elfdump
Error: Orphaned: bin/tutorial
Error: Orphaned: bin/write_obj
Error: Orphaned: bin/writer
===> Checking for items in pkg-plist which are not in STAGEDIR
===> Error: Plist issues found.

Maybe set

CONFIGURE_ARGS=        --bindir=${EXAMPLESDIR}

instead.

This revision now requires changes to proceed.Apr 14 2019, 6:45 PM

Updated Makefile based on suggestions;

Reinstated #CreatedBy header

Fixed install directory of examples binaries with CONFIGURE_ARGS= --bindir=${EXAMPLESDIR}

Now installs examples source code in addition to the example binaries

This revision is now accepted and ready to land.Apr 15 2019, 8:01 AM
This revision was automatically updated to reflect the committed changes.