[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Change by STINNER Victor : -- keywords: +patch pull_requests: +9931 stage: -> patch review ___ Python tracker ___ ___

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: Sorry for bumping this thread but Victor, could you please share your inputs on this if you have the time for it, thanks. -- ___ Python tracker

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Karthikeyan Singaravelan added the comment: >> The only thing here is that '-I' returns '-s -E -I' unlike other options >> where args can be used for comparison logic in check_options. > Karthikeyan, do you happen to have a use case where this might come into > action? I don't have a use

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: You're right Karthikeyan, although I personally think that returning ['-s', '-E', '-I'] should be a plausible solution here since it has been stated explicitly that it implies '-s' and '-E' but I'm still waiting for what Victor has to say on this. > The

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Karthikeyan Singaravelan added the comment: > From what I understand, this can be done in one of two ways. First, we could > edit > https://github.com/python/cpython/blob/ad73a9cf97770023665a1bb1c6390a3c99478139/Modules/main.c#L430 > and not incrementing -s and -E. But I believe this would

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: > I expect to get: ['-I'] instead of ['-s', '-E', '-I'] >From what I understand, this can be done in one of two ways. First, we could >edit >https://github.com/python/cpython/blob/ad73a9cf97770023665a1bb1c6390a3c99478139/Modules/main.c#L430 > and not

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: Linking this[1] here in case someone else stumbles upon this thread. I've created an issue and a PR for the documentation issue regarding the absence of -I flag from the sys.flags table which came into picture from the discussions in this thread. [1]:

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: Thank you Karthikeyan, I'm going to take care of both of these issues. -- ___ Python tracker ___

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Karthikeyan Singaravelan added the comment: > Thanks for bringing this up Karthikeyan, however, could there be another > reason why -I would be left out. Also, have you filed an issue for this? I couldn't see any related issue for this though the table was changed in 3.7.0 > Also, Victor

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Danish Prakash added the comment: > With respect to documentation I was talking about '-I' not being documented > in the table at https://docs.python.org/3.7/library/sys.html#sys.flags though > it's present in the C code and in sys.flags.isolated. Thanks for bringing this up Karthikeyan,

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

STINNER Victor added the comment: > ./python.exe -I -c 'import subprocess; > print(subprocess._args_from_interpreter_flags())' > ['-s', '-E', '-I'] This looks wrong, I would prefer to only get ['-I']. -- ___ Python tracker

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Karthikeyan Singaravelan added the comment: Thanks Victor for the details. > In the C code, sys.flags.isolated clearly documented as linked to the -I > option: With respect to documentation I was talking about '-I' not being documented in the table at

[issue34812] [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag

Change by STINNER Victor : -- keywords: +easy title: support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag -> [EASY] support.args_from_interpreter_flags() doesn't inherit -I (isolated) flag ___ Python tracker