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

Corrige make help sur MacOS #6454

Merged
merged 2 commits into from
Jan 21, 2023

Conversation

philippemilink
Copy link
Member

  • head n'est pas la version GNU sur MacOS, il ne supporte pas un nombre négatif de lignes
  • column affiche (null) pour les lignes vides

Contrôle qualité

Lancer make help et constater que l'affichage est correct. Je n'ai pas testé sur un Linux (mais il faudrait le faire).

@coveralls
Copy link

coveralls commented Jan 20, 2023

Coverage Status

Coverage: 88.302%. Remained the same when pulling 0b98d63 on philippemilink:fix-makefile-help into fe00164 on zestedesavoir:dev.

Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA NOK La commande ne fonctionne plus sous Linux (Ubuntu Budgie 20.04 LTS précisément) :

Use 'make [command]' to run one of these commands:

tail : option invalide -- 'r'
tail : option invalide -- 'r'
Saisissez « tail --help » pour plus d'informations.
Saisissez « tail --help » pour plus d'informations.

Open this Makefile to see what each command does.

Je ne pense que l'on va réussir à trouver une commande qui fonctionne sur les deux donc il va probablement falloir détecter l'OS et lancer la bonne commande en fonction. Soit en regardant la valeur de $OSTYPE (lien) ou de uname (autre lien) je suppose.

- head ne supporte pas un nombre négatif de lignes
- column affiche "(null)" pour les lignes vides
@philippemilink
Copy link
Member Author

Je viens de pusher une version qui prend en compte les différences entre un système GNU (Linux) ou un système Darwin (Mac). J'ai une légère différence d'affichage pour les titres :

  • sur Mac :
     ~ General
    install-linux                   Install the minimal components needed
    install-linux-full              Install all the components needed
    
  • sur Linux :
                                    ~ General
    install-linux                   Install the minimal components needed
    install-linux-full              Install all the components needed
    

Mais on va s'en contenter :)

Makefile Show resolved Hide resolved
Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je viens de tester la commande make help sur ma distribution Linux (Ubuntu Budgie 20.04 LTS) et l'affichage est correct (il n'y a pas le léger décalage que tu mentionnais plus tôt) ! Je te fais confiance pour le fonctionnement sur Mac.

@Situphen Situphen enabled auto-merge (rebase) January 21, 2023 21:00
@Situphen Situphen merged commit 1a70d23 into zestedesavoir:dev Jan 21, 2023
@philippemilink philippemilink deleted the fix-makefile-help branch January 21, 2023 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants