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

Make: Fix target name for Windows platform #7370

Merged
merged 1 commit into from Apr 25, 2023

Conversation

Biswa96
Copy link
Contributor

@Biswa96 Biswa96 commented Apr 25, 2023

This fixes the taget name by adding .exe extension for Windows platform.
Otherwise, the following error is shown with `make bin/gh.exe' command.

make: *** No rule to make target 'script/build', needed by 'bin/gh.exe'.  Stop.

This fixes the taget name by adding .exe extension for Windows platform.
Otherwise, the following error is shown with `make bin/gh.exe' command.

make: *** No rule to make target 'script/build', needed by 'bin/gh.exe'.  Stop.
@Biswa96 Biswa96 requested a review from a team as a code owner April 25, 2023 18:54
@Biswa96 Biswa96 requested review from samcoe and removed request for a team April 25, 2023 18:54
@cliAutomation cliAutomation added the external pull request originating outside of the CLI core team label Apr 25, 2023
@cliAutomation
Copy link
Collaborator

Hi! Thanks for the pull request. Please ensure that this change is linked to an issue by mentioning an issue number in the description of the pull request. If this pull request would close the issue, please put the word 'Fixes' before the issue number somewhere in the pull request body. If this is a tiny change like fixing a typo, feel free to ignore this message.

@cliAutomation cliAutomation added this to Needs review 🤔 in The GitHub CLI Apr 25, 2023
@Biswa96 Biswa96 changed the base branch from trunk to deployment April 25, 2023 18:55
@Biswa96 Biswa96 changed the title Fix makefile Make: Fix target name for Windows platform Apr 25, 2023
Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

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

Thank you!

The GitHub CLI automation moved this from Needs review 🤔 to Needs to be merged 🎉 Apr 25, 2023
@mislav mislav merged commit 3d1b40d into cli:deployment Apr 25, 2023
5 checks passed
The GitHub CLI automation moved this from Needs to be merged 🎉 to Pending Release 🥚 Apr 25, 2023
@Biswa96 Biswa96 deleted the fix-makefile branch April 26, 2023 03:22
mislav pushed a commit that referenced this pull request May 9, 2023
This fixes the taget name by adding .exe extension for Windows platform.
Otherwise, the following error is shown with `make bin/gh.exe' command.

    make: *** No rule to make target 'script/build', needed by 'bin/gh.exe'.  Stop.
jtpetty pushed a commit that referenced this pull request May 22, 2023
This fixes the taget name by adding .exe extension for Windows platform.
Otherwise, the following error is shown with `make bin/gh.exe' command.

    make: *** No rule to make target 'script/build', needed by 'bin/gh.exe'.  Stop.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external pull request originating outside of the CLI core team
Projects
No open projects
The GitHub CLI
  
Pending Release 🥚
Development

Successfully merging this pull request may close these issues.

None yet

3 participants