build: distribute variant templates content (#88848) #556

Merged
fpeters merged 1 commits from wip/88848-variant-symlinks into main 2024-03-29 15:34:04 +01:00
Owner

setuptools fait un filtrage filter(os.path.isfile, results) qui élimine les liens symboliques vers les répertoires. (mais pas totalement templates/variants/boussu-2022/combo/wcs/ est également un lien symbolique et celui-là est bien suivi, je n'ai pas tout compris).

Bref j'ajoute une passe pour rattraper ça, pour le cas précis des liens symboliques dans templates/variants/

setuptools fait un filtrage `filter(os.path.isfile, results)` qui élimine les liens symboliques vers les répertoires. (mais pas totalement templates/variants/boussu-2022/combo/wcs/ est également un lien symbolique et celui-là est bien suivi, je n'ai pas tout compris). Bref j'ajoute une passe pour rattraper ça, pour le cas précis des liens symboliques dans templates/variants/
fpeters added 1 commit 2024-03-29 11:08:38 +01:00
gitea/publik-base-theme/pipeline/head This commit looks good Details
6847e93ec0
build: distribute variant templates content (#88848)
fpeters changed title from build: distribute variant templates content (#88848) to WIP: build: distribute variant templates content (#88848) 2024-03-29 11:18:03 +01:00
Author
Owner

Ça n'est pas tout à fait ça, en fait setuptools conserve une première copie (exemple on a Hellemmes, qui est un lien symbolique vers Lille, mais pas Lomme, qui est également le même lien symbolique). Et pour Grenoble c'est Eybens qui est bon, mais pas grenoble ou grenoble-metropole-2022.

Ça n'est pas tout à fait ça, en fait setuptools conserve une première copie (exemple on a Hellemmes, qui est un lien symbolique vers Lille, mais pas Lomme, qui est également le même lien symbolique). Et pour Grenoble c'est Eybens qui est bon, mais pas grenoble ou grenoble-metropole-2022.
fpeters force-pushed wip/88848-variant-symlinks from 6847e93ec0 to bec5b85a0b 2024-03-29 12:24:53 +01:00 Compare
fpeters reviewed 2024-03-29 12:51:02 +01:00
@ -43,1 +44,4 @@
def local_findall(self, dir=os.curdir):
# override default file search to allow for duplicates (as some themes have
# multiple variants that are just symlinks)
Author
Owner

C'est la méthode findall de setuptools qui supprime les "doublons", cette fonction remplace ça.

C'est la méthode findall de setuptools qui supprime les "doublons", cette fonction remplace ça.
@ -44,0 +60,4 @@
def copy_file(self, *args, **kwargs):
# do not allow hardlinking as hardlinks would get inserted as is in
# the tarball, and would not extract properly.
kwargs.pop('link', None)
Author
Owner

Au moment de la création du répertoire qui va servir à la construction de l'archive, il ne fait pas qu'il y ait d'hardlinks (sinon ils se trouvent comme ça dans le tar.gz et à la décompression ça ne donne pas quelque chose de correct).

Au moment de la création du répertoire qui va servir à la construction de l'archive, il ne fait pas qu'il y ait d'hardlinks (sinon ils se trouvent comme ça dans le tar.gz et à la décompression ça ne donne pas quelque chose de correct).
fpeters changed title from WIP: build: distribute variant templates content (#88848) to build: distribute variant templates content (#88848) 2024-03-29 13:34:12 +01:00
csechet approved these changes 2024-03-29 15:31:06 +01:00
fpeters merged commit bec5b85a0b into main 2024-03-29 15:34:04 +01:00
fpeters deleted branch wip/88848-variant-symlinks 2024-03-29 15:34:04 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-base-theme#556
No description provided.