SCons: Do not print a warning when passing profile to command-line

[Xrayez]

The profile option is used to specify default SCons options via file. However, after #55203, this prints a warning when the option is used, because it's unrecognized by SCons.

Warning: Adding profile as an actual SCons option is not a solution, because it's used as a base for Variables(). therefore ignoring this option when printing a warning is enough.

For those not familiar with profile, see also #38821.

Bugsquad edit: This closes #56359.

[Calinou]

This could also be extended to Mono-specific SCons options, which are currently considered to be unknown.

[Xrayez]

This could also be extended to Mono-specific SCons options, which are currently considered to be unknown.

I looked into this, looks like the problem is that Variables() may be populated per-module, so this will affect all modules not just Mono. The potential solution could be to require to define get_opts() for each module similarly how it's done per platform, create a single instance of Variables() when all options are collected.

This is certainly outside the scope of this PR and requires some major refactor, due to this I think that #55203 should be reverted in 3.x branch, this affects my projects like: https://github.com/goostengine/goost, https://github.com/Xrayez/godot-anl etc.

Hardcoding values into the list as in this PR would also be quite hacky and error-prone, I don't recommend it. Again, this is not only about built-in modules, #55203 affects custom modules as well.

[akien-mga]

See #53030 which aimed to address that for Mono options while implementing the same feature.

I agree that hardcoding "not yet known" parameters is not a good option as it doesn't scale well for custom modules. I'll revert the change for 3.4 and 3.x until we figure out the best approach.

[Xrayez]

Thanks, note that I think this PR is still needed since profile option is a special case, so this could be merged regardless of further rewrite/refactor to collect modules options.

But it's up to you.

[Xrayez]

I noticed #56359 popped up, so yeah I recommend merging this PR.

[akien-mga]

I prefer to revert the change for now as there are still regressions to solve beside the special case of profile.
#56551

[Xrayez]

This is an excellent decision.