Skip to content

fix clone targets#1772

Closed
MartinNowak wants to merge 1 commit intodlang:stablefrom
MartinNowak:fix_clone_targets
Closed

fix clone targets#1772
MartinNowak wants to merge 1 commit intodlang:stablefrom
MartinNowak:fix_clone_targets

Conversation

@MartinNowak
Copy link
Member

  • broken by 4129e45 which changed target folders from
    ../druntime-2.074.1 to $PWD/druntime-2.074.1
  • no need to make those targets absolute since the makefile will
    always run in the dlang.org root folder

- broken by 4129e45 which changed target folders from
  ../druntime-2.074.1 to $PWD/druntime-2.074.1
- no need to make those targets absolute since the makefile will
  always run in the dlang.org root folder
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @MartinNowak!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Copy link
Contributor

@wilzbach wilzbach left a comment

Choose a reason for hiding this comment

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

LGTM, sorry for causing yet another outage at the nightly build server. I might submit a short follow-up with #1771 (comment)

@wilzbach
Copy link
Contributor

I might submit a short follow-up with #1771 (comment)

It seems that you directly need this without #1771

@MartinNowak
Copy link
Member Author

I see, needs to be incorporated with #1771 due to $PWD in the DMD_DIR et.al. vars.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants