Opened 11 years ago

Closed 5 years ago

#38768 closed enhancement (fixed)

atlas: use compiler_blacklist_versions portgroup

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: Veence (Vincent)
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc: cooljeanius (Eric Gallager), petrrr, maehne (Torsten Maehne)
Port: atlas

Description

The atlas port has this block of code in the +clang variant:

    if {[vercmp $xcodeversion 4.3] < 0} {
        pre-fetch {
            return -code error "Your version of clang is too old!"
        }
    }

This is not the correct way to check for this. You need to check the clang version, not the Xcode version, because the user could have updated Xcode but forgotten to update the command line tools; this happens all the time because users do not know that they need to actually open the Xcode application in order for the command line tools to get updated.

Use the compiler_blacklist_versions portgroup to blacklist the specific range of Xcode clang versions that will not work for atlas. If the clang is older, then MacPorts will use llvm-gcc-4.2 instead. XcodeVersionInfo lists some known clang versions.

It's odd to have a +clang variant at all and it seems like you could remove it. MacPorts will automatically choose to use clang on Xcode 4.2 and later anyway.

Change History (5)

comment:1 Changed 11 years ago by cooljeanius (Eric Gallager)

Cc: egall@… added

Cc Me!

comment:2 Changed 11 years ago by Veence (Vincent)

Ok Ryan. As soon as the other bugs are fixed, I’ll enhance the Portfile according to your hint.

comment:3 Changed 10 years ago by petrrr

Cc: Peter.Danecek@… added

Cc Me!

comment:4 Changed 7 years ago by maehne (Torsten Maehne)

Cc: maehne added

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

Resolution: fixed
Status: newclosed

The atlas port no longer contains this block.

Note: See TracTickets for help on using tickets.