ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Fr. Br. George" <george@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] [SCM] m/mathomatic/mathomatic.spec
Date: Fri, 8 Nov 2019 09:18:50 +0300
Message-ID: <20191108061850.GA10869@imap.altlinux.org> (raw)
In-Reply-To: <2b10068e-c032-1f2a-2037-f4df9fabe3d2@basealt.ru>

Зайчики и белочки! Пожалуйста, после того, как применили 2to3,
тестируйте работоспособность всех исправленных файлов.

2to3 не делает из программы на python2 работающей программы на python3,
она только исправлает некоторые очевидные синтаксические различия.

Если вы запилили без теста модуль для python3 в параллель к модулю на
Python2, ок, вы просто немножко всех обманули, отправили в Сизиф
сгенерённый роботом код, который никто никогда не запускал. Можно хотя
бы воспользоваться пакетом для python2.

Но если вы таким путём хотите получить _вместо_ программы на python2
программу на python3, без теста это уже практически диверсия.

On Thu, Nov 07, 2019 at 11:17:39PM +0300, Stanislav Levin wrote:
> 
> 07.11.2019 21:21, Vladimir D. Seleznev пишет:
>> ^ Этот способ не работает.

> Ниже перечислены изменения сделанные утилитой 2to3 в данном пакете.
Ссылки на git было бы достаточно.

> Очевидно, что изменения верные.
Странно было бы пользоватсья инструментом, который делает заведомо
неверные изменения.

> Можете сравнить их, например, с Fedora:
> https://src.fedoraproject.org/rpms/mathomatic/tree/master
Скорее всего, там тоже потрудились роботы.

> Что именно в данном случае не работает?
Не рабтает ни одна из пропатченных утилит.

>> 2to3 в большинстве случаев _не_ генерирует рабочий код.
> 
> Это утверждение требующее каких-то доказательств.
Например, 2to3 не заменяет string.split(строка) на строка.split()
И не факт, что это всегда возможно, надо почитать.

> Список всех багов по 2to3:
По ссылке не ходил. Но если там есть ваши исправления относительно
подобных случаев, то огромное вам спасибо, жить станет легче!

> Да, бывают нюансы. Но бОльшую массу кода трансформирует и трансформирует
> в рабочий Python3 код.
Это утверждение, требующее каких-то доказательств.

>> после каждого
>> перевода их надо проверять.
> Безусловно (если есть понимание изменений), потому что утилита не
> охватывает весь функциональный спектр.
Собственно, именно и только это является предметом нашего разговора.

-- 
	Fr. Br. George


  reply	other threads:[~2019-11-08  6:18 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-07 18:21 ` Vladimir D. Seleznev
2019-11-07 20:17   ` Stanislav Levin
2019-11-08  6:18     ` Fr. Br. George [this message]
2019-11-08  7:12       ` Stanislav Levin
2019-11-08 18:24         ` [devel] [JT] " Fr. Br. George
2019-11-08  7:26       ` [devel] " Anton Farygin
2019-11-08 18:26         ` Fr. Br. George
2019-11-09  7:01           ` Anton Farygin
2019-11-15 23:04               ` [devel] О переводе модулей на python3 Dmitry V. Levin
2019-11-17 18:30                 ` Anton Farygin
2019-11-17 18:30               ` Anton Farygin
2019-11-17 21:19                 ` Andrey Savchenko
2019-11-18  3:52                   ` Anton Farygin
2019-11-18  4:11                     ` Andrey Savchenko
2019-11-18  6:33                       ` Anton Farygin
2019-11-18  4:09               ` Anton Farygin
2019-11-18  4:52                 ` Антон Мидюков
2019-11-18  6:39                   ` Anton Farygin
2019-11-18  7:17                     ` Антон Мидюков
2019-11-18  7:49                 ` Denis Medvedev
2019-11-18  7:53                   ` Anton Farygin
2019-11-18 11:18                     ` Dmitry V. Levin
2019-11-18 11:30                       ` Anton Farygin
2019-11-18 11:22                 ` Dmitry V. Levin
2019-11-18 11:32                   ` Anton Farygin
2019-11-18 12:25                     ` Michael Shigorin
2019-11-18 12:40                     ` Grigory Ustinov
2019-11-18 12:45                       ` Andrey Bychkov
2019-11-18 16:06                       ` Anton Farygin
2019-11-18 15:24                   ` Vladimir D. Seleznev
2019-11-18  6:48               ` [devel] О переводе модулей на python3 (was: [SCM] m/mathomatic/mathomatic.spec) Stanislav Levin
2019-11-18 13:07                 ` Andrey Savchenko
2019-11-18 13:14                   ` [devel] qt3 Dmitry V. Levin
2019-11-18 16:03                   ` [devel] О переводе модулей на python3 (was: [SCM] m/mathomatic/mathomatic.spec) Alexey V. Vissarionov
2019-11-18 16:08                     ` [devel] О переводе модулей на python3 Anton Farygin
2019-11-08 18:48         ` [devel] [SCM] m/mathomatic/mathomatic.spec Fr. Br. George
2019-11-11 14:34             ` Fr. Br. George

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20191108061850.GA10869@imap.altlinux.org \
    --to=george@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git