Re: [Rpm-maint] [rpm-software-management/rpm] Python macro improvements (#221)

2017-08-28 Thread Ville Skyttä
scop commented on this pull request.



> @@ -0,0 +1,27 @@
+#!/usr/bin/python -tt

Shouldn't really matter all that much, because the script is invoked with an 
explicit "outside" interpreter without the shebang affecting anything, but 
meh... dropped.

-- 
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/221#discussion_r135551002___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Python macro improvements (#221)

2017-08-28 Thread Ville Skyttä
@scop pushed 1 commit.

9e192ce  python-macro-helper: Drop -tt from shebang


-- 
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/rpm-software-management/rpm/pull/221/files/f4ee9662cf5c0a04e1928487f1480eccc6d598b1..9e192cef134dd149e7813e7fc93bfa193ee95e4a
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Python macro improvements (#221)

2017-08-28 Thread ニール・ゴンパ
Conan-Kudo requested changes on this pull request.



> @@ -0,0 +1,27 @@
+#!/usr/bin/python -tt

Please don't use `-tt`, as that switch is deprecated and removed in Python 3.

-- 
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/221#pullrequestreview-58971161___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Python macro improvements (#221)

2017-08-28 Thread Ville Skyttä
Rebased. @pmatilai ping?

-- 
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/221#issuecomment-325380081___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Parametric macro arguments are not expanded (#127)

2017-08-28 Thread Panu Matilainen
Impossible to say without specific information. While most things can be 
written in a backwards compatible way, it's entirely possible some things 
cannot. I'd leave the older versions alone...

-- 
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/issues/127#issuecomment-325302818___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint