Jump to page: 1 2
Thread overview
[Issue 19014] Compiler imports symbols that aren't actually imported.
Jun 22, 2018
Mike Franklin
Jul 02, 2018
RazvanN
Jul 02, 2018
Mike Franklin
Jul 02, 2018
RazvanN
Jul 03, 2018
Mike Franklin
Jul 03, 2018
RazvanN
Jul 03, 2018
Mike Franklin
Jul 03, 2018
RazvanN
Dec 10, 2018
Mike Franklin
June 22, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

Steven Schveighoffer <schveiguy@yahoo.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |schveiguy@yahoo.com

--- Comment #1 from Steven Schveighoffer <schveiguy@yahoo.com> ---
I think this has something to do with intrinsics.

--
June 22, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #2 from Mike Franklin <slavo5150@yahoo.com> ---
Compiling with head I get the following:

Deprecation: module core.stdc.math is not accessible here, perhaps add static import core.stdc.math;

So, I believe this issue will be solved when the deprecation phase has passed.

--
July 02, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |razvan.nitu1305@gmail.com

--- Comment #3 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to Mike Franklin from comment #2)
> Compiling with head I get the following:
> 
> Deprecation: module core.stdc.math is not accessible here, perhaps add static import core.stdc.math;
> 
> So, I believe this issue will be solved when the deprecation phase has passed.

So, can we close this?

--
July 02, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #4 from Mike Franklin <slavo5150@yahoo.com> ---
I don't want to close it until a test is added to the DMD test suite and it passes.  We can't do that until the deprecation period has expired.

--
July 02, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #5 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to Mike Franklin from comment #4)
> I don't want to close it until a test is added to the DMD test suite and it passes.  We can't do that until the deprecation period has expired.

A compilable test with the REQUIRED_ARGS set to -de can be added in order to solve this.

--
July 03, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #6 from Mike Franklin <slavo5150@yahoo.com> ---
This issue is currently blocking https://github.com/dlang/druntime/pull/2222

I don't want to close this until that change is possible.  It's not currently possible because the compiler does not emit an error for the scenario in Comment #0.

After the deprecation period has expired, we can emit a compiler error, and then we can continue with https://github.com/dlang/druntime/pull/2222

I don't want to close this until https://github.com/dlang/druntime/pull/2222 can be implemented.

--
July 03, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #7 from RazvanN <razvan.nitu1305@gmail.com> ---
https://github.com/dlang/dmd/pull/8443 : turns deprecation into error

--
July 03, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #8 from Mike Franklin <slavo5150@yahoo.com> ---
(In reply to RazvanN from comment #7)
> https://github.com/dlang/dmd/pull/8443 : turns deprecation into error

Thank you, I tested locally, and that appears to resolve this issue.  I believe this can be closed when that is merged.  For some reason I though we had to wait a year before changing that to an error.

--
July 03, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #9 from RazvanN <razvan.nitu1305@gmail.com> ---
(In reply to Mike Franklin from comment #8)
> (In reply to RazvanN from comment #7)
> > https://github.com/dlang/dmd/pull/8443 : turns deprecation into error
> 
> Thank you, I tested locally, and that appears to resolve this issue.  I believe this can be closed when that is merged.  For some reason I though we had to wait a year before changing that to an error.

Normally, the deprecation period is one year, but in this case the deprecation was introduced in february 2016 (2 years and 4 months ago).

--
December 09, 2018
https://issues.dlang.org/show_bug.cgi?id=19014

--- Comment #10 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/00b990f8f96903a1fc6264c678fef6a58542f0c2 Fix Issue 19014 - Compiler imports symbols that aren't actually imported

https://github.com/dlang/dmd/commit/3d8663595c143e925195b9eec9cea96f88c8d213 Merge pull request #9050 from JinShil/fix_19014

Fix Issue 19014 - Compiler imports symbols that aren't actually imported merged-on-behalf-of: Nicholas Wilson <thewilsonator@users.noreply.github.com>

--
« First   ‹ Prev
1 2