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

fix Issue 19140 - AssertError@dmd/ctfeexpr.d(229): Assertion failure #8537

Merged
merged 2 commits into from Aug 6, 2018

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Aug 5, 2018

https://issues.dlang.org/show_bug.cgi?id=19140

When compiling with -verrors=spec

(spec:1) test/compilable/interpret3.d(7751): Error: f19140 cannot be interpreted at compile time, because it has no available source code
(spec:1) Error: cannot implicitly convert expression <error> of type void to real

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
19140 regression [REG master] AssertError@dmd/ctfeexpr.d(229): Assertion failure

⚠️⚠️⚠️ Warnings ⚠️⚠️⚠️

To target stable perform these two steps:

  1. Rebase your branch to upstream/stable:
git rebase --onto upstream/stable upstream/master
  1. Change the base branch of your PR to stable

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "master + dmd#8537"

@ibuclaw ibuclaw added the Regression PRs that fix regressions label Aug 5, 2018
@RazvanN7
Copy link
Contributor

RazvanN7 commented Aug 5, 2018

Please re-target stable.

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 5, 2018

No, because this is a regression only in master.

@ibuclaw
Copy link
Member Author

ibuclaw commented Aug 5, 2018

Also, I'll be rebasing gdc against master on a weekly basis, so all these implementation regressions will be rooted out long before release. ;-)

@wilzbach
Copy link
Member

wilzbach commented Aug 6, 2018

@CyberShadow any idea why DAutoTest is now permanently failing?
The logs only show this:

dmd/dinifile.d(99): Error: file `"SYSCONFDIR.imp"` cannot be found or not in a path specified with `-J`

...

posix.mak:469: recipe for target '../generated/linux/release/64/dmd' failed
make: *** [../generated/linux/release/64/dmd] Error 1

@wilzbach wilzbach merged commit e15d4df into dlang:master Aug 6, 2018
@CyberShadow
Copy link
Member

@CyberShadow any idea why DAutoTest is now permanently failing?

It doesn't look like a permanent failure judging from the branch history:
http://dtest.dlang.io/history/master

Perhaps bad dependencies in the makefile causing an unstable build order?

@ibuclaw ibuclaw deleted the issue19140 branch August 6, 2018 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Bug Fix Regression PRs that fix regressions
Projects
None yet
6 participants