Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makefile - établissement de la liste des fichiers modifiés #1411

Open
christopheNan opened this issue Sep 11, 2020 · 1 comment
Open

Makefile - établissement de la liste des fichiers modifiés #1411

christopheNan opened this issue Sep 11, 2020 · 1 comment
Labels

Comments

@christopheNan
Copy link
Collaborator

@christopheNan christopheNan commented Sep 11, 2020

La ligne 137 du Makefile recense les fichiers modifiés par rapport à la branche en cours.
Le problème est que cette branche n'a pas forcément été importée sur le fork personnel car CONTRIBUTING ne l'indique pas explicitement. On obtient donc :

[user@machine] make 
fatal: argument '3.9' ambigu : révision inconnue ou chemin inexistant.
Utilisez '--' pour séparer les chemins des révisions, comme ceci :
'git <commande> [<révision>...] -- [<chemin>...]'

Le reste du Makefile ne fonctionne pas correctement car la liste n'est pas bonne.

Propositions :

  1. forcer la comparaison avec le dépôt upstream, mais ça fait un accès réseau potentiellement (souvent) inutile ;
  2. faire la vérification que la branche existe localement et, si non, arrêter le build ou revenir à la solution 1 ;
  3. ajouter dans CONTRIBUTING qu'il faut git pull upstream la branche courante.
@christopheNan christopheNan added the meta label Sep 11, 2020
@christopheNan christopheNan changed the title Makefile non conforme à CONTRIBUTING Makefile - établissement de la liste des fichiers modifiés Sep 12, 2020
@christopheNan
Copy link
Collaborator Author

@christopheNan christopheNan commented Sep 12, 2020

Une autre proposition est de modifier la manière dont est construite la liste des fichiers modifiés.
Actuellement : git diff --name-only $(BRANCH) | grep '.po$$'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.