Inconsistent behavior of `zipapp.create_archive`
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 77.2k
- Fork
- 35.9k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Bug report
Bug description:
Currently, zipapp.create_archive makes the target application archive executable only if target is a str. (see snippets bellow).
I think the target application archive should be made executable also when target is os.PathLike.
# source.py
print("Hello from pyz")
# bug_reproduction.py
import os
import pathlib
import zipapp
import sys
zipapp.create_archive("source.py", "source.pyz")
zipapp.create_archive(
pathlib.Path("source.pyz"),
pathlib.Path("copy.pyz"),
interpreter=sys.executable,
)
zipapp.create_archive("source.pyz", "copy2.pyz", interpreter=sys.executable)
assert os.access("copy2.pyz", os.X_OK) # Passes
assert os.access("copy.pyz", os.X_OK) # Fails
CPython versions tested on:
CPython main branch, commit b0c9fc37dd0ed60456313600e6a01478308f195e
Operating systems tested on:
Linux
Linked PRs
- gh-156569
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Parti dall’entry point zipapp.create_archive e riproduci su Linux il comportamento descritto nell’issue con target string e pathlib.Path. Il lavoro è completato quando i target PathLike producono archivi eseguibili in modo coerente con i target string; il PR collegato nell’issue indica che questo lavoro è già in corso.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 25/100