New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD #17699
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice change. Tested locally, works well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your changes are good to have for such cases 👍.
Thanks for this.
Currently if supplying multiple arguments for ADDONS_TO_BUILD that include a regex eg. ADDONS_TO_BUILD='game.libretro.* peripheral.joystick' The regex will not match or be processed
4ff7e2e
to
5600eb2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
[cmake] addons support multiple regex arguments for ADDONS_TO_BUILD
Description
Currently if supplying multiple arguments for ADDONS_TO_BUILD that include a regex
The regex will not match or be processed at all.
You currently need to explicitly set each if you include multiple arguments
Motivation and Context
Playing with libretro addons, found that a regex isnt used if supplied in a multi argument addons_to_build
How Has This Been Tested?
Mac OS building multiple combinations of selective arguments
exact match of game.libretro and peripheral.joystick, regex match of game.libretro.mrboom, game.libretro.2048
exact match of game.libretro and peripheral.joystick, regex match of game.libretro.mrboom, game.libretro.2048
Exact match of pvr.hdhomerun and peripheral.joystick, with regex match of pvr.argustv
This also highlights an existing behaviour regarding the core game.libretro. You cannot selectively only build game.libretro as all other game.libretro.* automatically match with the cmake if check. This behaviour does not change with this PR.
xbmc/cmake/addons/CMakeLists.txt
Line 259 in 2d7e977
I think the easier way to resolve this would be to rename game.libretro to something like game.libretro.core. I couldnt come up with anything easy to handle this case, and game.libretro is the only case i could find this happens to across any binary addons. No other active binary addon, has the name that is a subset of other active addons
Screenshots (if appropriate):
Types of change
Checklist: