Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-27 Thread Panu Matilainen
Okay so related to the platform-python weirdness. That's a Fedora invention and it's up to Fedora to patch around it if needed. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-27 Thread Panu Matilainen
Merged #341. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/rpm-software-management/rpm/pull/341#event-1313715435___ Rpm-maint mailing list

Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-27 Thread Igor Gnatenko
@pmatilai I had concern about this just because Python maintainers asked me to use only `-s` for DNF.. > ... I suggest to add -s flag to the shebang until platform-python thing is > resolved. This will prevent troubles caused by dnf being affected by Python > packages installed by "sudo pip"

Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-25 Thread Ville Skyttä
> I don't think that passing -E is good Could you elaborate on that a bit, especially when you think -s /is/ good? Without -s, code from user site packages is loaded, potentially interfering with the commands. Without -E, PYTHONPATH from environment can be used to do exactly the same. (If

Re: [Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-25 Thread Igor Gnatenko
ignatenkobrain commented on this pull request. I don't think that passing -E is good (while -s is definitely good idea). We are planning to do same for dnf, the only -s part... As requested by Python maintainers -- You are receiving this because you are subscribed to this thread. Reply to

[Rpm-maint] [rpm-software-management/rpm] Invoke python-macro-helper with -Es python args (#341)

2017-10-25 Thread Ville Skyttä
To limit environment and user home dir influence. You can view, comment on, or merge this pull request online at: https://github.com/rpm-software-management/rpm/pull/341 -- Commit Summary -- * Invoke python-macro-helper with -Es python args -- File Changes -- M macros.in (6) M