[devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Slava Semushin]

Здравствуйте!

Решил обновить flawfinder до новой версии (#10747, ещё пару месяцев и
ей будет год). Так вот у меня вопрос возник, зачем был нужен и нужен
ли сейчас следующий патч:

[c0der@rock ~/flawfinder]$ cat flawfinder-0.22-alt-python.patch
--- flawfinder-0.22/flawfinder~ Wed May  8 01:37:02 2002
+++ flawfinder-0.22/flawfinder  Wed May 15 12:30:44 2002
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/python

 # flawfinder: Find potential security flaws ("hits") in source code.
 # Usage:
[c0der@rock ~/flawfinder]$


2ldv@: в бОльшей степени жду ответа от вас, как от мэйнтейнера пакета :)


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 797 bytes --]

On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin wrote:
> Решил обновить flawfinder до новой версии (#10747, ещё пару месяцев и
> ей будет год). Так вот у меня вопрос возник, зачем был нужен и нужен
> ли сейчас следующий патч:
> 
> [c0der@rock ~/flawfinder]$ cat flawfinder-0.22-alt-python.patch
> --- flawfinder-0.22/flawfinder~ Wed May  8 01:37:02 2002
> +++ flawfinder-0.22/flawfinder  Wed May 15 12:30:44 2002
> @@ -1,4 +1,4 @@
> -#!/usr/bin/env python
> +#!/usr/bin/python
> 
>  # flawfinder: Find potential security flaws ("hits") in source code.
>  # Usage:
> [c0der@rock ~/flawfinder]$

Хотите школьную аналогию?  Зачем нужно упрощать выражения?

> 2ldv@: в бОльшей степени жду ответа от вас, как от мэйнтейнера пакета :)

Это я сейчас исправлю.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Slava Semushin]

2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin wrote:
[...]
> > [c0der@rock ~/flawfinder]$ cat flawfinder-0.22-alt-python.patch
> > --- flawfinder-0.22/flawfinder~ Wed May  8 01:37:02 2002
> > +++ flawfinder-0.22/flawfinder  Wed May 15 12:30:44 2002
> > @@ -1,4 +1,4 @@
> > -#!/usr/bin/env python
> > +#!/usr/bin/python
[...]
> Хотите школьную аналогию?  Зачем нужно упрощать выражения?

Я не очень понял, что вы хотели сказать мне этими двумя вопросами.

Неясно зачем было менять шабанг, если итак всё работает. Или в 2002 не
работало? Короче, я хочу выкинуть этот патч, так как не вижу в нём
смысла.


> > 2ldv@: в бОльшей степени жду ответа от вас, как от мэйнтейнера пакета :)
> Это я сейчас исправлю.

Если собираетесь перевешивать на nobody@, то перевешивайте сразу на меня :)


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Vitaly Lipatov]

On 5 ноября 2007, Slava Semushin wrote:
> 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
...
> > > -#!/usr/bin/env python
> > > +#!/usr/bin/python
>
> [...]
>
> > Хотите школьную аналогию?  Зачем нужно упрощать выражения?
>
> Я не очень понял, что вы хотели сказать мне этими двумя
> вопросами.
Похоже это было намерение упростить выражение x + x до 2 * x :)

> Неясно зачем было менять шабанг, если итак всё работает. Или в
Более того, согласно полиси, должно было быть именно
#!/usr/bin/env python


-- 
С уважением,
Виталий Липатов
Санкт-Петербург
GNU! ALT Linux Team! WINE! LaTeX! LyX! http://freesource.info

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 858 bytes --]

On Tue, Nov 06, 2007 at 12:55:35AM +0300, Vitaly Lipatov wrote:
> On 5 ноября 2007, Slava Semushin wrote:
> > 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
> ...
> > > > -#!/usr/bin/env python
> > > > +#!/usr/bin/python
> >
> > [...]
> >
> > > Хотите школьную аналогию?  Зачем нужно упрощать выражения?
> >
> > Я не очень понял, что вы хотели сказать мне этими двумя
> > вопросами.
> Похоже это было намерение упростить выражение x + x до 2 * x :)

Cкорее x + 0 + y -> x + y.

> > Неясно зачем было менять шабанг, если итак всё работает. Или в
> Более того, согласно полиси, должно было быть именно
> #!/usr/bin/env python

Это неправильно.
Если в действующем полиси это написано, значит надо исправить.
Вы ещё
#!/usr/bin/env env env python
напишите.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Kirill A. Shutemov]

[-- Attachment #1: Type: text/plain, Size: 1359 bytes --]

On [Tue, 06.11.2007 01:21], Dmitry V. Levin wrote:
> On Tue, Nov 06, 2007 at 12:55:35AM +0300, Vitaly Lipatov wrote:
> > On 5 ноября 2007, Slava Semushin wrote:
> > > 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > > > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
> > ...
> > > > > -#!/usr/bin/env python
> > > > > +#!/usr/bin/python
> > >
> > > [...]
> > >
> > > > Хотите школьную аналогию?  Зачем нужно упрощать выражения?
> > >
> > > Я не очень понял, что вы хотели сказать мне этими двумя
> > > вопросами.
> > Похоже это было намерение упростить выражение x + x до 2 * x :)
> 
> Cкорее x + 0 + y -> x + y.
> 
> > > Неясно зачем было менять шабанг, если итак всё работает. Или в
> > Более того, согласно полиси, должно было быть именно
> > #!/usr/bin/env python
> 
> Это неправильно.

Почему не правильно? А если питон вдруг в /bin переложат? Или
/usr/local/bin?

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + Velesys LLC, http://www.velesys.com/
 + ALT Linux Team, http://www.altlinux.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch? [-]

[Michael Shigorin]

On Tue, Nov 06, 2007 at 01:21:08AM +0300, Dmitry V. Levin wrote:
> > > Неясно зачем было менять шабанг, если итак всё работает.
> > Более того, согласно полиси, должно было быть именно
> > #!/usr/bin/env python
> Это неправильно.  Если в действующем полиси это написано,
> значит надо исправить.

А мне кажется, что _это_ и исправления не стоит.
Питон -- не та штука, где скосить просмотр каталогов
что-то заметное сэкономит.  Их ещё много будет ;)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 832 bytes --]

On Tue, Nov 06, 2007 at 12:09:49PM +0200, Kirill A. Shutemov wrote:
> On [Tue, 06.11.2007 01:21], Dmitry V. Levin wrote:
> > On Tue, Nov 06, 2007 at 12:55:35AM +0300, Vitaly Lipatov wrote:
> > > On 5 ноября 2007, Slava Semushin wrote:
> > > > 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > > > > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
> > > ...
> > > > > > -#!/usr/bin/env python
> > > > > > +#!/usr/bin/python
[...]
> > > > Неясно зачем было менять шабанг, если итак всё работает. Или в
> > > Более того, согласно полиси, должно было быть именно
> > > #!/usr/bin/env python
> > 
> > Это неправильно.
> 
> Почему не правильно? А если питон вдруг в /bin переложат? Или
> /usr/local/bin?

Если переложат, то сделают симлинк, поскольку /usr/bin/python много где
используется.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch? [-]

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 727 bytes --]

On Tue, Nov 06, 2007 at 01:52:58PM +0200, Michael Shigorin wrote:
> On Tue, Nov 06, 2007 at 01:21:08AM +0300, Dmitry V. Levin wrote:
> > > > Неясно зачем было менять шабанг, если итак всё работает.
> > > Более того, согласно полиси, должно было быть именно
> > > #!/usr/bin/env python
> > Это неправильно.  Если в действующем полиси это написано,
> > значит надо исправить.
> 
> А мне кажется, что _это_ и исправления не стоит.

В полиси даже опечатка стОит исправления, тем более ошибка.

> Питон -- не та штука, где скосить просмотр каталогов
> что-то заметное сэкономит.  Их ещё много будет ;)

Я не призываю везде заменять "env python" на "python" прямо сейчас.
Есть более актуальные задачи.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1181 bytes --]

On Tue, Nov 06, 2007 at 02:24:30PM +0200, Kirill A. Shutemov wrote:
> On [Tue, 06.11.2007 15:19], Dmitry V. Levin wrote:
> > On Tue, Nov 06, 2007 at 12:09:49PM +0200, Kirill A. Shutemov wrote:
> > > On [Tue, 06.11.2007 01:21], Dmitry V. Levin wrote:
> > > > On Tue, Nov 06, 2007 at 12:55:35AM +0300, Vitaly Lipatov wrote:
> > > > > On 5 ноября 2007, Slava Semushin wrote:
> > > > > > 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > > > > > > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
> > > > > ...
> > > > > > > > -#!/usr/bin/env python
> > > > > > > > +#!/usr/bin/python
> > [...]
> > > > > > Неясно зачем было менять шабанг, если итак всё работает. Или в
> > > > > Более того, согласно полиси, должно было быть именно
> > > > > #!/usr/bin/env python
> > > > 
> > > > Это неправильно.
> > > 
> > > Почему не правильно? А если питон вдруг в /bin переложат? Или
> > > /usr/local/bin?
> > 
> > Если переложат, то сделают симлинк, поскольку /usr/bin/python много где
> > используется.
> 
> Так а почему собственно не правильно использовать /usr/bin/evn python ?
> Для ruby я всегда так делаю.

Потому что не переложат. :)


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch?

[Kirill A. Shutemov]

[-- Attachment #1: Type: text/plain, Size: 1436 bytes --]

On [Tue, 06.11.2007 15:19], Dmitry V. Levin wrote:
> On Tue, Nov 06, 2007 at 12:09:49PM +0200, Kirill A. Shutemov wrote:
> > On [Tue, 06.11.2007 01:21], Dmitry V. Levin wrote:
> > > On Tue, Nov 06, 2007 at 12:55:35AM +0300, Vitaly Lipatov wrote:
> > > > On 5 ноября 2007, Slava Semushin wrote:
> > > > > 2007/11/6, Dmitry V. Levin <ldv / altlinux.org>:
> > > > > > On Tue, Nov 06, 2007 at 12:01:05AM +0600, Slava Semushin
> > > > ...
> > > > > > > -#!/usr/bin/env python
> > > > > > > +#!/usr/bin/python
> [...]
> > > > > Неясно зачем было менять шабанг, если итак всё работает. Или в
> > > > Более того, согласно полиси, должно было быть именно
> > > > #!/usr/bin/env python
> > > 
> > > Это неправильно.
> > 
> > Почему не правильно? А если питон вдруг в /bin переложат? Или
> > /usr/local/bin?
> 
> Если переложат, то сделают симлинк, поскольку /usr/bin/python много где
> используется.

Так а почему собственно не правильно использовать /usr/bin/evn python ?
Для ruby я всегда так делаю.

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + Velesys LLC, http://www.velesys.com/
 + ALT Linux Team, http://www.altlinux.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

Re: [devel] Q: Нужен ли flawfinder-0.22-alt-python.patch? [-]

[Michael Shigorin]

On Tue, Nov 06, 2007 at 03:21:59PM +0300, Dmitry V. Levin wrote:
> > > > > Неясно зачем было менять шабанг, если итак всё работает.
> > > > Более того, согласно полиси, должно было быть именно
> > > > #!/usr/bin/env python
> > > Это неправильно.  Если в действующем полиси это написано,
> > > значит надо исправить.
> > А мне кажется, что _это_ и исправления не стоит.
> В полиси даже опечатка стОит исправления, тем более ошибка.

А, ну это разумеется.

Только тогда там стоит озвучить как SHOULD, не MUST, IMHO...

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/