dlang.org icon indicating copy to clipboard operation
dlang.org copied to clipboard

Fix building dlang.org from scratch

Open wilzbach opened this issue 6 years ago • 7 comments

See: https://forum.dlang.org/post/[email protected]

The problem was that GIT_HOME was undefined and thus the git clone failed. I think we should really have a CI for this. @CyberShadow what do you think about re-enabling Travis?

I also added a hint about DIFFABLE=1 to the docs while I was at it.

wilzbach avatar Apr 13 '18 12:04 wilzbach

Thanks for your pull request, @wilzbach!

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.

dlang-bot avatar Apr 13 '18 12:04 dlang-bot

@CyberShadow what do you think about re-enabling Travis?

Why was it disabled?

I don't know if it's going to bring enough benefit vs. the overhead of managing another CI. I guess we could try and see if it doesn't get in the way too much?

CyberShadow avatar Apr 13 '18 12:04 CyberShadow

I just tried "make -f posix.mak all" again. It still fails for me, after:" CC="c++" /tmp/.host_dmd-2.072.2/dmd2/osx/bin/dmd -conf=/tmp/.host_dmd-2.072.2/dmd2/osx/bin/dmd.conf -of../generated/osx/release/64/dmd -m64 -vtls -J../generated/osx/release/64 -J../res -L-lstdc++ -version=MARS -fPIC -w -de dmd/access.d ..... .......................(Long list of files)................. sed -i -e "s|../druntime/import |../druntime-2.079.0/import |" -e "s|../phobos |../phobos-2.079.0 |" .generated/dmd-2.079.0/generated/osx/release/64/dmd.conf sed: -e: No such file or directory make: *** [.generated/dmd-2.079.0/generated/osx/release/64/dmd] Error 1 make: *** Deleting file `.generated/dmd-2.079.0/generated/osx/release/64/dmd' "

zachthemystic avatar Apr 14 '18 05:04 zachthemystic

Looks like your sed isn't GNU sed.

CyberShadow avatar Apr 14 '18 10:04 CyberShadow

Looks like your sed isn't GNU sed.

This link might help: https://stackoverflow.com/questions/2320564/i-need-my-sed-i-command-for-in-place-editing-to-work-with-both-gnu-sed-and-bsd

zachthemystic avatar Apr 14 '18 12:04 zachthemystic

Why was it disabled?

I think because we moved the testsuite to DAutoTest to keep the CI overhead low here.

I don't know if it's going to bring enough benefit vs. the overhead of managing another CI. I guess we could try and see if it doesn't get in the way too much?

Ok enabled. Ideally it should just work "out of the box" (as for any fresh checkout).

Looks like your sed isn't GNU sed.

Ok. I added a "Posix-compatible" way with using a temporary file. Not sure why we need to build this compiler anyhow, Maybe we can even remove the dependence on this entirely, but for now "... && mv" should work.

wilzbach avatar Apr 14 '18 21:04 wilzbach

Build failed. But at least we know that it ran for 15 minutes and 50 seconds before doing so!

zachthemystic avatar Apr 16 '18 14:04 zachthemystic