Project

General

Profile

Bug #47476

make clean - nettoyage des css

Added by Lauréline Guérin over 2 years ago. Updated about 2 years ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
08 October 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

#1

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.

#2

Updated by Nicolas Roche over 2 years ago

Fait en me basant sur le .gitignore.

#3

Updated by Thomas Noël over 2 years ago

en se basant sur gitignore... donc plutôt "git clean -fdx" ?

#5

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.

#6

Updated by Frédéric Péters over 2 years ago

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.

#7

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.

#8

Updated by Lauréline Guérin about 2 years ago

  • Assignee set to Frédéric Péters

Also available in: Atom PDF