Thread overview
[Issue 10327] New: Missing 'package.d' for DIP37 needs a better error message
Jun 10, 2013
Andrej Mitrovic
Jun 10, 2013
Andrej Mitrovic
Jun 10, 2013
Andrej Mitrovic
Jul 21, 2013
Walter Bright
June 10, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=10327

           Summary: Missing 'package.d' for DIP37 needs a better error
                    message
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Keywords: diagnostic
          Severity: normal
          Priority: P2
         Component: DMD
        AssignedTo: nobody@puremagic.com
        ReportedBy: andrej.mitrovich@gmail.com


--- Comment #0 from Andrej Mitrovic <andrej.mitrovich@gmail.com> 2013-06-10 15:40:37 PDT ---
If you have the directory "foo\bar" and try to compile this test.d file:

-----
module test;

import foo.bar;

void main() { }
-----

The error is: test.d(3): Error: module bar is in file 'foo\bar\package.d' which
cannot be read

When there is no such directory the error is simple:

The error is: test.d(3): Error: module bar is in file 'foo\bar.d' which cannot
be read

This is ok.

But the package error message is confusing to anyone not aware of the feature in DIP37. To make it informative, it should print:

test.d(3): Error: importing package foo.bar requires package.d which is not found in 'foo\bar\package.d'

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
June 10, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=10327



--- Comment #1 from Andrej Mitrovic <andrej.mitrovich@gmail.com> 2013-06-10 15:42:03 PDT ---
(In reply to comment #0)
> test.d(3): Error: importing package foo.bar requires package.d which is not found in 'foo\bar\package.d'

Or "which cannot be found" or "which cannot be read".

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
June 10, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=10327


Andrej Mitrovic <andrej.mitrovich@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull
         AssignedTo|nobody@puremagic.com        |andrej.mitrovich@gmail.com


--- Comment #2 from Andrej Mitrovic <andrej.mitrovich@gmail.com> 2013-06-10 16:27:26 PDT ---
https://github.com/D-Programming-Language/dmd/pull/2159

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
July 21, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=10327



--- Comment #3 from github-bugzilla@puremagic.com 2013-07-21 15:00:11 PDT ---
Commits pushed to master at https://github.com/D-Programming-Language/dmd

https://github.com/D-Programming-Language/dmd/commit/0d30f12f2b9f0b4fe71938b02065a752ebe6fb63 Fixes Issue 10327 - More informative error message on missing package.d file.

https://github.com/D-Programming-Language/dmd/commit/477e42a03576d758c8902afb495525d84c0a1d1e Merge pull request #2159 from AndrejMitrovic/Fix10327

Issue 10327 - More informative error message on missing package.d file

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
July 21, 2013
http://d.puremagic.com/issues/show_bug.cgi?id=10327


Walter Bright <bugzilla@digitalmars.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |bugzilla@digitalmars.com
         Resolution|                            |FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------