[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #12 from github-bugzi...@puremagic.com --- Commits pushed to stable at https://github.com/D-Programming-Language/druntime https://github.com/D-Programming-Language/druntime/commit/28e3bb192f76de535ad8bc430667567584affc05 fix Issue 14750 - druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again https://github.com/D-Programming-Language/druntime/commit/6bc37c057379aeea0ac1fae5c5436f144772 Merge pull request #1323 from WalterBright/fix14750 --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #11 from github-bugzi...@puremagic.com --- Commits pushed to master at https://github.com/D-Programming-Language/druntime https://github.com/D-Programming-Language/druntime/commit/28e3bb192f76de535ad8bc430667567584affc05 fix Issue 14750 - druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again https://github.com/D-Programming-Language/druntime/commit/6bc37c057379aeea0ac1fae5c5436f144772 Merge pull request #1323 from WalterBright/fix14750 [reg] fix Issue 14750 - druntime/test/coverage was added to druntime, but n… --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 github-bugzi...@puremagic.com changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 Jonathan M Davis issues.dl...@jmdavisprog.com changed: What|Removed |Added CC||issues.dl...@jmdavisprog.co ||m --- Comment #9 from Jonathan M Davis issues.dl...@jmdavisprog.com --- (In reply to Walter Bright from comment #7) True, you can do that (thanks for the tip!) but the zip file is for use on systems that don't have git. But does it need to be generated on any systems without git? I would have thought that it would be fine for the target for the zip file to only work on systems with git installed, because the primary purpose of the zip file is to create it for distributing a release, and the releases are being done by developers who need git to work on the main D repos in the first place. --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #10 from Walter Bright bugzi...@digitalmars.com --- (In reply to Jonathan M Davis from comment #9) But does it need to be generated on any systems without git? Yes. I do it all the time. That's how I find out when it breaks. --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #7 from Walter Bright bugzi...@digitalmars.com --- True, you can do that (thanks for the tip!) but the zip file is for use on systems that don't have git. --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #8 from Walter Bright bugzi...@digitalmars.com --- https://github.com/D-Programming-Language/druntime/pull/1323 --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #6 from Martin Nowak c...@dawg.eu --- (In reply to Walter Bright from comment #5) It fails because the files necessary are not there. What's the point of having a MANIFEST if it is only a subset of the manifest? Exactly, let's delete MANIFEST. The question is why you need it or the zip rule. We could replace the rule with git archive --format=zip HEAD druntime.zip or zip druntime.zip $(git ls-files) depending on whether or not modified working dir files should end up in the zip. --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #5 from Walter Bright bugzi...@digitalmars.com --- (In reply to Martin Nowak from comment #4) (In reply to Walter Bright from comment #3) I'm fine if someone posts a method of doing this automatically. In the absence of such, it needs to be done manually. Otherwise, things break, as they did for me. We don't use the MANIFEST file to generate the source trees for the releases and all the last releases already contain the test folder. So what exactly was your problem? -- make -f win32.mak zip ... copy druntime.zip file to new system ... unzip druntime.zip make -fposix.mak MODEL=32 clean make -C test/init_fini clean rm -rf obj make -C test/exceptions clean rm -rf obj make[1]: *** No rule to make target `test/coverage/.clean', needed by `clean'. Stop make: *** [clean] Error 2 --- It fails because the files necessary are not there. What's the point of having a MANIFEST if it is only a subset of the manifest? --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #4 from Martin Nowak c...@dawg.eu --- (In reply to Walter Bright from comment #3) I'm fine if someone posts a method of doing this automatically. In the absence of such, it needs to be done manually. Otherwise, things break, as they did for me. We don't use the MANIFEST file to generate the source trees for the releases and all the last releases already contain the test folder. So what exactly was your problem? --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #1 from Walter Bright bugzi...@digitalmars.com --- In fact, it seems none of the files in the test directory are in the manifest. --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 Martin Nowak c...@dawg.eu changed: What|Removed |Added CC||c...@dawg.eu --- Comment #2 from Martin Nowak c...@dawg.eu --- (In reply to Walter Bright from comment #0) When adding files to druntime, PLEASE add them to the manifest. Please stop recommending repeated manual actions as solutions, they don't work. (In reply to Walter Bright from comment #1) In fact, it seems none of the files in the test directory are in the manifest. Why should we distribute the additional tests? --
[Issue 14750] druntime/test/coverage was added to druntime, but not to the MANIFEST - zip file broken again
https://issues.dlang.org/show_bug.cgi?id=14750 --- Comment #3 from Walter Bright bugzi...@digitalmars.com --- (In reply to Martin Nowak from comment #2) (In reply to Walter Bright from comment #0) When adding files to druntime, PLEASE add them to the manifest. Please stop recommending repeated manual actions as solutions, they don't work. I'm fine if someone posts a method of doing this automatically. In the absence of such, it needs to be done manually. Otherwise, things break, as they did for me. There's no point in having a MANIFEST file if it is allowed to get out of sync with the files. Why should we distribute the additional tests? Because running 'make -f posix.mak clean' fails without them. That's how I discovered the problem. --
