Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#63738 closed defect (fixed)

auto-multiple-choice/auto-multiple-choice-devel: fix build failures for older macOS releases, or set known_fail

Reported by: mascguy (Christopher Nielsen) Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version: 2.7.1
Keywords: Cc: nortcele
Port: auto-multiple-choice auto-multiple-choice-devel

Description (last modified by mascguy (Christopher Nielsen))

Presently these two ports are failing to build for older macOS releases, causing a significant amount of wasted time on the buildbots. (Note that the port itself builds quickly. Instead, it's the installation of dependencies which takes the bulk of the time.)

However, this might be easily fixable, perhaps with an appropriate compiler blacklist.

Here's one example, for 10.7:

https://build.macports.org/builders/ports-10.7_x86_64-builder/builds/72714/steps/install-port/logs/stdio

gcc -o AMC-buildpdf AMC-buildpdf.cc  -O2 -I/opt/local/include -I. -DNEEDS_GETLINE -std=c++11 -L/opt/local/lib   -lstdc++ -lm -I/opt/local/include/pango-1.0 -I/opt/local/include/harfbuzz -I/opt/local/include/pango-1.0 -I/opt/local/include/fribidi -I/opt/local/include/harfbuzz -I/opt/local/include/poppler/glib -I/opt/local/include/poppler -I/opt/local/include/cairo -I/opt/local/include/glib-2.0 -I/opt/local/lib/glib-2.0/include -I/opt/local/include/pixman-1 -I/opt/local/include/ossp -I/opt/local/include/freetype2 -I/opt/local/include/libpng16 -L/opt/local/lib -lpangocairo-1.0 -lpango-1.0 -lharfbuzz -lpoppler-glib -lgobject-2.0 -lglib-2.0 -lintl -lcairo -I/opt/local/include/opencv4 -L/opt/local/lib/opencv4 -lopencv_gapi.4.5.0 -lopencv_highgui.4.5.0 -lopencv_ml.4.5.0 -lopencv_objdetect.4.5.0 -lopencv_photo.4.5.0 -lopencv_stitching.4.5.0 -lopencv_video.4.5.0 -lopencv_calib3d.4.5.0 -lopencv_features2d.4.5.0 -lopencv_flann.4.5.0 -lopencv_videoio.4.5.0 -lopencv_imgcodecs.4.5.0 -lopencv_imgproc.4.5.0 -lopencv_core.4.5.0

cc1plus: error: unrecognized command line option "-std=c++11"cc1plus: error: unrecognized command line option "-std=c++11"

But if the blacklist option doesn't work, we should set known_fail for older releases, and save precious buildbot time.

Change History (11)

comment:1 Changed 3 years ago by evanmiller (Evan Miller)

Have you tried compiler.cxx_standard 2011?

comment:2 in reply to:  1 Changed 3 years ago by mascguy (Christopher Nielsen)

Replying to evanmiller:

Have you tried compiler.cxx_standard 2011?

That would certainly be preferable to blacklisting!

comment:3 Changed 3 years ago by kencu (Ken)

Compiler selection is broken for this port:

gcc -o 

so nothing will work until that is repaired.

It is only serendipity that it builds at all on any system.

See UsingTheRightCompiler.

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

Description: modified (diff)

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

Upstream eschews any type of configure-related script, in favor of a makefile. However, the makefile portgroup might bridge the gap, in terms of the compiler setup.

Thankfully the makefile is downright simple and concise, relative to most software these days. They even provide a formal mechanism to select pre-baked configuration profiles, including one specific to MacPorts. Nice!

Net-Net, even if a patch is necessary, it should be very simple. And we'll certainly contribute back to upstream as well.

Presently testing on my 10.7 VM, to see what's needed.

Last edited 3 years ago by mascguy (Christopher Nielsen) (previous) (diff)

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

Owner: changed from nortcele to mascguy

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

Cc: nortcele added

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

Resolution: fixed
Status: assignedclosed

In 63b34f8bb0eb432d4baf7f57e158eafdbc16cd54/macports-ports (master):

auto-multiple-choice/auto-multiple-choice-devel: pass chosen compiler to makefile

  • Makefile references GCC/GCC_PP, rather than CXX. Fixes builds across-the-board.

Closes: #63738

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

In 55b25a2f83556ba9dfa0d234e31b346ee9199bb7/macports-ports (master):

auto-multiple-choice/auto-multiple-choice-devel: blacklist clang < 900
See: #63738

comment:10 Changed 3 years ago by nortcele

Thank you for solving the problem. I didn't have the skill to solve it.

comment:11 in reply to:  10 Changed 3 years ago by mascguy (Christopher Nielsen)

Replying to nortcele:

Thank you for solving the problem. I didn't have the skill to solve it.

You're welcome Joel!

If you ever need help with anything, don't hesitate to ask. We're always willing to provide assistance!

Last edited 3 years ago by mascguy (Christopher Nielsen) (previous) (diff)
Note: See TracTickets for help on using tickets.