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

Build 2.068+ without binary downloads #17

Closed
mihails-strasuns opened this issue Apr 4, 2015 · 8 comments
Closed

Build 2.068+ without binary downloads #17

mihails-strasuns opened this issue Apr 4, 2015 · 8 comments

Comments

@mihails-strasuns
Copy link
Contributor

Curently Digger tries to download binary 2.066.1 release from http://downloads.dlang.org/releases/2.x/2.066.1/dmd.2.066.1.zip to build DMD version that requires host D compiler. It is preferrable to have an option to either build it locally or use a system-wide installation.

@CyberShadow
Copy link
Owner

It is preferrable to have an option to either build it locally

At some point, building DMD will require a compiler newer than 2.067. Should Digger compile DMD three times in that case?

or use a system-wide installation.

This risks subtly wrong builds if e.g. the wrong version is installed. If Digger is to achieve its goal of completely isolated and reproducible builds, it needs control over all dependencies.

FWIW, the download will not occur if the ZIP file is placed in the /dl/ directory.

@mihails-strasuns
Copy link
Contributor Author

Should Digger compile DMD three times in that case?

As an option, yes. Automating bootstrapping feels to me one of major motivating points for usage of tools like Digger.

This risks subtly wrong builds if e.g. the wrong version is installed.

What about optionally defining it during compile-time? The thing is, I don't like to package things that silently download external binaries :) So some way to say "if you need DMD 2.067.0, just use system package" is very tempting path because I will maintain both in a lockstep. I am considering applying such distro-specific patch locally but wanted to hear your opinion first.

@CyberShadow
Copy link
Owner

cf1ae94 adds --bootstrap

@mihails-strasuns
Copy link
Contributor Author

Awesome, thanks! Is it included in latest tagged release already?

@CyberShadow
Copy link
Owner

No, it will be part of the next release.

On Wed, May 13, 2015 at 10:02 AM, Михаил Страшун notifications@github.com
wrote:

Awesome, thanks! Is it included in latest tagged release already?


Reply to this email directly or view it on GitHub
#17 (comment).

@mihails-strasuns
Copy link
Contributor Author

Will make an Arch Linux package as soon as it is out in that case.

@CyberShadow
Copy link
Owner

Uploaded 2.2 beta. Will publish 2.2 in a week barring any issues.

@CyberShadow
Copy link
Owner

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

No branches or pull requests

2 participants