Bug 10022 – Importing packages

Status
RESOLVED
Resolution
FIXED
Severity
enhancement
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2013-05-03T14:17:00Z
Last change time
2013-07-13T23:01:31Z
Keywords
pull
Assigned to
yebblies
Creator
yebblies

Comments

Comment #0 by yebblies — 2013-05-03T14:17:08Z
When an import resolves to a package, instead import a module inside the package called 'package'. Usual rules are then followed as if it was imported explicitly. When both a module and a package exist, the compiler gives an ambiguity error.
Comment #1 by yebblies — 2013-05-03T15:13:19Z
Comment #2 by andrej.mitrovich — 2013-05-03T15:24:52Z
Same as Issue 3603?
Comment #3 by yebblies — 2013-05-03T15:35:53Z
(In reply to comment #2) > Same as Issue 3603? No.
Comment #4 by issues.dlang — 2013-05-04T17:55:29Z
> Same as Issue 3603? No, it's a variant of DIP 15. It'll make it so that you could do something like replace std/datetime.d with std/datetime/common.d std/datetime/interval.d std/datetime/package.d std/datetime/timepoint.d std/datetime/timezone.d and std/datetime/pakckage.d could then look something like -------------- /++ Package documentation here +/ module std.datetime.package; public import std.datetime.common; public import std.datetime.interval; public import std.datetime.timepoint; public import std.datetime.timezone; -------------- So, std.datetime could then be broken up and no code would break. It would also likely replace the all.d idiom that some people have been using to make it so that you can import an entire package at once, since it's basically all.d with compiler support. Even better, it also then gives us a way to document packages. We discussed it with Andrei and Walter at dconf after Adam Wilson was complaining about the lack of this sort of ability in his talk, and we eventually got Andrei and Walter to agree to it.
Comment #5 by andrej.mitrovich — 2013-05-05T03:08:41Z
(In reply to comment #4) > -------------- > /++ Package documentation here +/ > module std.datetime.package; > > public import std.datetime.common; > public import std.datetime.interval; > public import std.datetime.timepoint; > public import std.datetime.timezone; > -------------- I see. This is a good idea, but public import modules do have their bugs, e.g. the recently fixed Issue 9919. I've ran into a few others over the years.
Comment #6 by yebblies — 2013-05-05T09:13:24Z
(In reply to comment #5) > > I see. This is a good idea, but public import modules do have their bugs, e.g. > the recently fixed Issue 9919. I've ran into a few others over the years. Bugs are just bugs, we will just have to fix them as they appear.
Comment #7 by issues.dlang — 2013-05-05T22:17:11Z
Comment #8 by bugzilla — 2013-06-05T15:12:24Z
Comment #9 by yebblies — 2013-07-13T23:01:31Z
This has been implemented.