bug-global
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Don't detect the Python shebang line at build-time; Don't use AM_PAT


From: Shigio YAMAGUCHI
Subject: Re: Don't detect the Python shebang line at build-time; Don't use AM_PATH_PYTHON
Date: Mon, 17 Apr 2023 21:33:11 +0900

I will value the following two things:
o GNU Standard
o Consistency of configure options of Global
  (--with-posix-sort, --with-universal-ctags and etc)

So I cannot accept your patch as is. I'll rewrite it with '--with-'
prefix when I have time, but until then It will be suspended.
Anyway, thank you for the valuable patch.

Regards,
Shigio

On Mon, Apr 17, 2023 at 8:53 PM Olivier Certner <ocert.dev@free.fr> wrote:
>
> Hi,
>
> > Wouldn't it be more appropriate to rename this option to
> > '--with-python-interpreter'?
>
> May be. Depends on your point of view.
>
> The letter of:
> https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.71/autoconf.html#Site-Configuration
> isn't clear on this specific situation.
>
> It is true that the '--with-' prefix is normally reserved for interfacing 
> with external packages, which would seem to be the case here. However, the 
> Pygments parser is not really "installed" (in a Python sense) and is always 
> provided, whether Python/Pygments are installed or not. And, for the record, 
> I prefer it like that: No hard dependencies, but if I have Python and 
> Pygments installed, I can use them.
>
> What the current option only does is simply modifying a shebang line, and 
> according to the text below '--with-package' in the document you cited:
> """Do not use a ‘--with’ option to specify the file name to use to find 
> certain files. That is outside the scope of what ‘--with’ options are for."""
>
> So perhaps the real GNU-compliant solution is just to declare an environment 
> variable that can override the default shebang.
>
> The same reasoning would apply to '--with-exuberant-ctags' and 
> '--with-universal-ctags' (and the DIR argument to '--with-sqlite3'). For 
> backwards compatibility and simplicity, I would understand that you prefer to 
> keep existing things as they are, and that you prefer 
> '--with-python-interpreter' for consistency with those.
>
> In the end, I don't think being that picky in this case matters that much, so 
> really I don't mind changing "--enable-python-interpreter" to 
> "--with-python-interpreter" or an environment variable.
>
> What do you prefer?
>
> Regards,
> Olivier
>
>
>


-- 
Shigio YAMAGUCHI <shigio@gnu.org>
PGP fingerprint:
26F6 31B4 3D62 4A92 7E6F  1C33 969C 3BE3 89DD A6EB



reply via email to

[Prev in Thread] Current Thread [Next in Thread]