Bug #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.
Files
History
Updated by Benjamin Dauvergne over 2 years ago
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.
Updated by Nicolas Roche over 2 years ago
- File 0001-makefile-remove-generated-files-on-make-clean-47476.patch 0001-makefile-remove-generated-files-on-make-clean-47476.patch added
- Tracker changed from Support to Bug
- Status changed from Nouveau to Solution proposée
- Assignee set to Nicolas Roche
- Patch proposed changed from No to Yes
Fait en me basant sur le .gitignore.
Updated by Thomas Noël over 2 years ago
en se basant sur gitignore... donc plutôt "git clean -fdx" ?
Updated by Nicolas Roche over 2 years ago
- Status changed from Solution proposée to En cours
- Assignee deleted (
Nicolas Roche)
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.
Updated by Frédéric Péters over 2 years ago
- File 0001-build-remove-generated-css-files-on-make-clean-47476.patch 0001-build-remove-generated-css-files-on-make-clean-47476.patch added
- Status changed from En cours to 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.
Updated by Frédéric Péters over 2 years ago
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.