Opened 3 years ago

Last modified 20 months ago

#63243 reopened defect

[boost17x] : python issues in the post-destroot

Reported by: RJVB (René Bertin) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc: michaelld (Michael Dickens), cooljeanius (Eric Gallager)
Port: boost171, boost176

Description

There are 2 issues related to Python in the port:boost17x ports:

  • the glob expression assumes that a python variant has always been set. This need not be the case, in which case glob will raise an error. Instead, glob -nocomplain should be used (as an alternative to tracking if a python variant has indeed been set).
  • I don't think it's appropriate to create libboost_python3-mt.dylib when using the +python27 variant.

Change History (8)

comment:1 Changed 3 years ago by mascguy (Christopher Nielsen)

Cc: mascguy removed
Owner: set to mascguy
Status: newassigned

comment:2 Changed 3 years ago by mascguy (Christopher Nielsen)

Thanks Rene, will commit the Python fix shortly.

comment:3 Changed 3 years ago by Christopher Nielsen <mascguy@…>

In c1068a9dc07b9bf28c3ef201cefdbfe12fd22bb2/macports-ports (master):

boostXXX: fix post-destroot failure when Python disabled
See: #63243

comment:4 Changed 3 years ago by michaelld (Michael Dickens)

Resolution: fixed
Status: assignedclosed

comment:5 Changed 3 years ago by mascguy (Christopher Nielsen)

Resolution: fixed
Status: closedreopened

Michael, reopening this, as Rene's second item hasn't been resolved yet. ("I don't think it's appropriate to create libboost_python3-mt.dylib when using the +python27 variant.")

Speaking of which, can you speak to that? Sounds reasonable to me, but wanted to get your thoughts.

comment:6 Changed 3 years ago by michaelld (Michael Dickens)

@macsguy My bad! I should have read through the issue more carefully!

comment:7 Changed 3 years ago by michaelld (Michael Dickens)

I'm in agreement to not create create libboost_python3-mt.dylib when using the +python27 variant ... that's just wrong!

comment:8 Changed 20 months ago by cooljeanius (Eric Gallager)

Cc: cooljeanius added
Note: See TracTickets for help on using tickets.