Opened 7 months ago

Last modified 7 months ago

#71318 assigned defect

boost: Remove this wrapper port; it keeps breaking different dependents of Boost

Reported by: barracuda156 Owned by: michaelld (Michael Dickens)
Priority: Normal Milestone:
Component: ports Version: 2.10.4
Keywords: Cc: mascguy (Christopher Nielsen), reneeotten (Renee Otten)
Port: boost

Description (last modified by barracuda156)

The wrapper port serves no good purpose, its only function is breaking various consumers of Boost libs.

Just another example: https://github.com/NickvisionApps/libnick/issues/50 And one more: https://github.com/macports/macports-ports/pull/25855 And we got yet more in ports tree.

These errors never show up on CI or buildbots, for obvious reasons, but the current set-up is wrong. It wastes time on debugging and then forces to add per-port hacks via conflicts_build.

Change History (8)

comment:1 Changed 7 months ago by barracuda156

Description: modified (diff)

comment:2 Changed 7 months ago by ryandesign (Ryan Carsten Schmidt)

Cc: mascguy added
Owner: set to michaelld
Status: newassigned
Summary: Remove `boost` wrapper port, it keeps breaking different dependents of Boostboost: Remove this wrapper port; it keeps breaking different dependents of Boost

Duplicate of, or at least dependent upon, #63136.

comment:3 in reply to:  2 ; Changed 7 months ago by mascguy (Christopher Nielsen)

Replying to ryandesign:

Duplicate of, or at least dependent upon, #63136.

Yes definitely, there are still a number of ports that depend on boost:

./editors/sigil-qt4/Portfile:depends_lib-append      port:boost \
./editors/textmate2/Portfile:                        port:boost \
./games/glob2/Portfile:depends_lib         port:boost \
./games/PlasmaClient/Portfile:                            port:boost \
./games/wesnoth/Portfile:depends_lib-append      port:boost \
./graphics/agave/Portfile:                port:boost
./graphics/alembic/Portfile:    depends_lib-append      port:boost
./graphics/exempi/Portfile:     depends_lib-append port:boost
./graphics/vtk/Portfile:    depends_lib-append port:boost \
./net/et/Portfile:    depends_lib-append port:boost
./python/py-pynds/Portfile:                        port:boost \
./science/aircraft_oap/Portfile:depends_lib-append  port:boost \
./science/lscsoft-deps/Portfile:        port:boost
./science/openbabel/Portfile:depends_lib-append  port:boost \
./science/plumed/Portfile:    depends_lib-append port:boost
./sysutils/QLColorCode/Portfile:depends_build       port:boost
./textproc/freeling/Portfile:                    port:boost
./www/FileZilla/Portfile:                    port:boost

comment:4 Changed 7 months ago by reneeotten (Renee Otten)

the issue with the “papillo” port in the PR you reference is that you didn’t supply the correct location to look for boost as configure arguments. I am not saying we can’t go without the “boost” wrapper port but often it’s pretty straightforward to resolve things without resorting to “conflicts_builds”.

comment:5 in reply to:  3 Changed 7 months ago by barracuda156

Replying to mascguy:

Replying to ryandesign:

Duplicate of, or at least dependent upon, #63136.

Yes definitely, there are still a number of ports that depend on boost:

Yes, I am aware of that. I used this in one of my ports earlier too. But they all should just use boost176 directly or, if possible, boost181.

comment:6 in reply to:  4 ; Changed 7 months ago by barracuda156

Replying to reneeotten:

the issue with the “papillo” port in the PR you reference is that you didn’t supply the correct location to look for boost as configure arguments. I am not saying we can’t go without the “boost” wrapper port but often it’s pretty straightforward to resolve things without resorting to “conflicts_builds”.

Boost portgroup should take care of finding the right boost. If it fails with papilo, then it is a PG issue.

comment:7 in reply to:  6 Changed 7 months ago by reneeotten (Renee Otten)

Cc: reneeotten added

Replying to barracuda156:

Boost portgroup should take care of finding the right boost. If it fails with papilo, then it is a PG issue.

No, it is not. Upstream can pick whatever random variable/configure argument they decide on so it’s impossible for any PortGroup to make that work all the time. It’s the port maintainers/person doing the update to take care of it.

comment:8 Changed 7 months ago by reneeotten (Renee Otten)

having said that, I am fine with removing the shim port and have only the versioned ones. Feel free to do the requiired work and submit a PR.

Note: See TracTickets for help on using tickets.