FS#71311 - [ninja] install ninja_syntax.py

Attached to Project: Community Packages
Opened by Thomas Weißschuh (t-8ch) - Sunday, 20 June 2021, 07:17 GMT
Last edited by Toolybird (Toolybird) - Sunday, 16 April 2023, 08:25 GMT
Task Type Feature Request
Category Packages
Status Closed
Assigned To Anatol Pomozov (anatolik)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

The upstream ninja developers provide a python module to generate ninja build files.
This module is located at misc/ninja_syntax.py .
Currently the Arch package does not ship that file.

There is an AUR package available ( https://aur.archlinux.org/packages/python-ninja-syntax/ ) but this uses an outdated and unofficial version of the file off of PyPI.

It would be nice if the ninja package could provide this module instead for both python 2 and 3.
This task depends upon

Closed by  Toolybird (Toolybird)
Sunday, 16 April 2023, 08:25 GMT
Reason for closing:  Fixed
Comment by Eli Schwartz (eschwartz) - Sunday, 20 June 2021, 15:52 GMT
  • Field changed: Task Type (Bug Report → Feature Request)
  • Field changed: Status (Unconfirmed → Assigned)
  • Task assigned to Anatol Pomozov (anatolik)
We're not going to install anything for python2 if it's not needed by an existing repo package, stop using python2 already...

The python3 version seems reasonable.
Comment by Thomas Weißschuh (t-8ch) - Sunday, 20 June 2021, 16:17 GMT
Fine by me. Was more like a proactive request.
Comment by Celeste (Coelacanthus) - Friday, 19 August 2022, 16:56 GMT
Hi Eli Schwartz,
since the python3 version is reasonable, could you package python files in the Arch ninja package?
Thanks.
Comment by Anatol Pomozov (anatolik) - Sunday, 21 August 2022, 01:41 GMT
I can add the script as

install -m644 -D misc/ninja_syntax.py "$pkgdir/usr/lib/python3.10/site-packages/ninja_syntax.py"

but I wonder if "$pkgdir/usr/lib/python3.10/site-packages" is the best place for the python script.
Comment by Thomas Weißschuh (t-8ch) - Sunday, 21 August 2022, 08:12 GMT
I have seen users that expect to be able to use `import ninja_syntax`.
Also it would be compatible to the AUR package.
Comment by Anatol Pomozov (anatolik) - Tuesday, 30 August 2022, 22:15 GMT
ninja-1.11.1-1 added an experimental ninja_syntax support. Please try it.
Comment by Thomas Weißschuh (t-8ch) - Thursday, 01 September 2022, 07:57 GMT
Looks good, thanks!

Loading...