Skip to content

Compile ignores /S/S/Scripts/CompatibilityList #48

Open
@Nuc1eoN

Description

@Nuc1eoN

I have written a new recipe At-Spi2-Core 2.54.0 which is supposed to replace ATK.

Therefore I have added the following to /S/S/Scripts/CompatibilityList.

...
ATK: At-Spi2-Core

In my understanding this is supposed to replace a dependency called ATK by At-Spi2-Core in a program's Dependency file.

However upon running eg Compile --debug ATKMM (which has ATK in its Dependencies file), Compile still sandboxes ATK rather than At-Spi2-Core:

$ sudo Compile --debug atkmm
...
adding dependency /Programs/ATK/2.38.0
...

Apprently this had been noticed before, but never got the final fix:

IRC log
2021-02-04
<paranoidd> ermo: mmm, I must have replaced Ncurses with NcursesW on some Dependencies file (and forgot
            to take note of which file I touched)
<paranoidd> anyhow, /S/Settings/Scripts/CompatibilityList should have been involved at some point to map
            Ncurses -> NcursesW
<paranoidd> /System/Settings/Scripts/CompatibilityList 25:Ncurses: NcursesW
<ermo> paranoidd: roger
<ermo> did you read my forum post?
<ermo> I still think the idea I proposed there might have merit in terms of being able to easily introspect
       recursive CheckDependencies calls?
<paranoidd> just read it. Yeah, that's a good idea! Let me implement it now, one sec
<ermo> :thumbsup:
<ermo> paranoidd: also, have you seen the bug I filed re. the kernel package and BIOS-only systems?
<ermo> on GH
<paranoidd> yes, I have. But I didn't have time to look into that one yet
<ermo> roger
<ermo> paranoidd: so, /S/S/S/CompatibilityList also has Ncurses: NcursesW here
<ermo> do you think it might presently be ignored?
<ermo> It's a pretty decent way of handling temporary renames (until all deps get updated)
<paranoidd> it's usually handled by CheckDependencies, but the branch the code is taking when we call it with
            --mode=list is probably causing it to skip looking into that file
<ermo> lemme know when you need me to try again
<ermo> box is at the ready ;)
<paranoidd> ermo: done. Please update your copies of Scripts and Compile
<ermo> on it
<paranoidd> I'll update the thread on the forum to state this change
<paranoidd> ah!
<paranoidd> on Compile, when we print the resolved dependencies, we're sorting the output alphabetically
<paranoidd> let me change that, otherwise this new feature becomes useless
<paranoidd> done. Please update Compile again
...
<ermo> paranoidd: no dice, Ncurses is still ignored...

Just for reference, these are the commits from the day of the discussion:
Compile:
6b4c9d1391b752263a241444d6d375f1dab7cf51
3e5865b99c93fa823172c41f1694a36b8d763e9b
Scripts:
adf1c6b

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions