Skip to content

Make: Fix target name for Windows platform#7370

Merged
mislav merged 1 commit intocli:deploymentfrom
Biswa96:fix-makefile
Apr 25, 2023
Merged

Make: Fix target name for Windows platform#7370
mislav merged 1 commit intocli:deploymentfrom
Biswa96:fix-makefile

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.

@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!

@mislav mislav merged commit 3d1b40d into cli:deployment 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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments