Page MenuHomeFreeBSD

emulators/i386-wine-devel: Style improvements
ClosedPublic

Authored by salvadore on Apr 23 2020, 9:45 PM.
Tags
None
Referenced Files
F153223861: D24550.id71075.diff
Sun, Apr 19, 10:01 PM
Unknown Object (File)
Sun, Apr 19, 4:10 AM
Unknown Object (File)
Thu, Apr 16, 7:33 PM
Unknown Object (File)
Thu, Apr 16, 6:10 AM
Unknown Object (File)
Tue, Apr 14, 5:02 PM
Unknown Object (File)
Wed, Apr 8, 11:27 PM
Unknown Object (File)
Mon, Apr 6, 8:10 PM
Unknown Object (File)
Mon, Apr 6, 12:17 PM
Subscribers

Details

Summary
  • Reorder variables.
  • Remove tools for updating pkg-plist: they were broken by precedent commits and based on the python 2 script files/mergeplist.py, but python 2 is now EOL.
  • Move port-update target from Makefile.amd64 to Makefile and rename it distinfo-update as it does not update pkg-plist anymore.
  • Remove EXTRACT_SUFX and use properly USES=tar:txz instead.
  • Remove references to removed wine staging ports in CONFLICTS_INSTALL.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Tested successfully.
It only needs a mentor approval now :)

lgtm

head/emulators/i386-wine-devel/Makefile.amd64
52

It's arguable, whether this is the right position for the options in the makefile.

This revision is now accepted and ready to land.Apr 27 2020, 5:48 PM
head/emulators/i386-wine-devel/Makefile.amd64
52

Where would you put them?
I put them there trying to follow the porter's handbook's indications: https://www.freebsd.org/doc/en/books/porters-handbook/porting-order.html

head/emulators/i386-wine-devel/Makefile.amd64
52

My gut feeling would have put them above the block where you set the PLIST_SUBs.

But I would not lose any sleep over it :)

This revision was automatically updated to reflect the committed changes.

Committed, thanks tcberner for your comments!