Development #47476
make clean - nettoyage des css
0%
Description
Lorsqu'on exécute un make clean, les css compilées à partir de scss ne devraient-elles pas être supprimées ?
Cela ne semble pas être le cas actuellement.
Fichiers
Demandes liées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 3 ans
Pour apporter plus de détails, lors du développement du ticket #47474 Lauréline s'est aperçu qu'après modification des .scss pour retirer edit-profile
et recompilation des CSS via make all
certaines CSS n'étaient pas mise à jour et pas non plus supprimées via make clean
; ne sachant pas quoi faire je lui ai conseillé un rm `git ls-files -o`
en attendant puis d'ouvrir ce ticket pour voir si quelque chose manquait dans le Makefile.
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Fichier 0001-makefile-remove-generated-files-on-make-clean-47476.patch 0001-makefile-remove-generated-files-on-make-clean-47476.patch ajouté
- Tracker changé de Support à Bug
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Nicolas Roche
- Patch proposed changé de Non à Oui
Fait en me basant sur le .gitignore
.
Mis à jour par Thomas Noël il y a plus de 3 ans
en se basant sur gitignore... donc plutôt "git clean -fdx" ?
Mis à jour par Nicolas Roche il y a plus de 3 ans
- Statut changé de Solution proposée à En cours
- Assigné à
Nicolas Rochesupprimé
Mince j'ai mal regardé.
dist: clean
La cible
dist
appelle clean
et donc il ne faut pas que cette dernière efface les CSS et les PNG.
Oui, je pense que pour nettoyer, taper "git clean -fdX" à la main convient mieux ici.
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Fichier 0001-build-remove-generated-css-files-on-make-clean-47476.patch 0001-build-remove-generated-css-files-on-make-clean-47476.patch ajouté
- Statut changé de En cours à Solution proposée
Je pense que le problème plus important serait l'absence de recompilation mentionnée en premier commentaire mais je ne constate pas ça, je modifie le includes/_forms.scss et je tape make css et toutes les intégrations me semblent bien recompilées.
Quant à compliquer les choses à la façon de ton patch, on peut quand même être assez sûr que ça ne sera pas maintenu. Si on veut quelque chose, je propose le plus simple patch attaché et de s'arrêter à ça.
Mis à jour par Frédéric Péters il y a plus de 3 ans
Et inclure ou pas des artefacts préconstruits dans le tarball, c'est selon; clairement dans un tarball classique tu ne trouves pas les .o à côté des .c; donc pour moi ce n'est pas une question à se poser.
Mis à jour par Robot Gitea il y a 11 mois
- Tracker changé de Bug à Development
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/277
- Titre : build: remove generated css files on make clean (#47476)
- Modifications : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/277/files
Mis à jour par Robot Gitea il y a 11 mois
- Statut changé de Solution proposée à Solution validée
Corentin Sechet (csechet) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 11 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/277
- Titre : build: remove generated css files on make clean (#47476)
- Modifications : https://git.entrouvert.org/entrouvert/publik-base-theme/pulls/277/files
Mis à jour par Transition automatique il y a 11 mois
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a 5 mois
- Lié à Bug #8508: nettoyer "make sdist" de publik-base-theme ajouté