Page MenuHomeFreeBSD

Merge net/unison-nox11 as a flavor of net/unison
ClosedPublic

Authored by madpilot on Jan 6 2018, 9:43 AM.
Tags
None
Referenced Files
F82873264: D13783.id37996.diff
Fri, May 3, 10:35 AM
F82873261: D13783.id.diff
Fri, May 3, 10:35 AM
F82873258: D13783.id37654.diff
Fri, May 3, 10:35 AM
F82873254: D13783.id37584.diff
Fri, May 3, 10:35 AM
F82873249: D13783.id37953.diff
Fri, May 3, 10:35 AM
Unknown Object (File)
Sun, Apr 28, 7:30 AM
Unknown Object (File)
Fri, Apr 19, 2:14 PM
Unknown Object (File)
Mar 18 2024, 9:45 PM
Subscribers

Details

Summary

I'd like to merge the nox11 port as a flavor of the main port.

Later I'd do the same for the older versions of unison in net/unison232 and net/unison240

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14373
Build 14528: arc lint + arc unit

Event Timeline

madpilot added a reviewer: portmgr. madpilot added 1 blocking reviewer(s): bsam.Jan 6 2018, 9:45 AM

I'm all for the change.
Thank you for taking care of those ports!

mat requested changes to this revision.Jan 8 2018, 2:36 PM
mat added inline comments.
net/unison/Makefile
18

Please add:

FLAVOR?=  ${FLAVORS[1]}

here. It makes all the flavors testing alter easier.

57

.else would be enough.

This revision now requires changes to proceed.Jan 8 2018, 2:36 PM
MOVED
9846

add flavor to second field.

madpilot marked 3 inline comments as done.

Patch including requested changes.

First patch also missed PLIST_SUB variables, due to a mistake.

You don't need all of the :U constructs any more.

Removed the :U constructs.

While here I also noticed stage-qa reported some missing dependencies which I added.

Looks ok for me.

net/unison/Makefile
22–24

You can simply use unison and unison-nox11 here, without globs to match versions.

This revision is now accepted and ready to land.Jan 15 2018, 3:05 PM
net/unison/Makefile
22–24

I was sticking to this since that was the recommended way, but a test shows pkg is smart enough not to trip itself so I'll use the simpler syntax.

Thanks for pointing this out.

net/unison/Makefile
22–24

Right, I have to rewrite the "conflicts handling" section of the handbook, thanks for pointing that out :-)

This revision was automatically updated to reflect the committed changes.