Re: Relaxin the PVP with regards to adding instances

On Tue, Sep 11, 2012 at 6:14 AM, Michael Sloan wrote:

Glad to hear that you agree that it's consistent! This would make the PVP simpler.

I've been thinking, there's still a way to retain the property that you can specify version bounds which retain compilation, even with orphan instances. If your package really needs to add an instance to a datatype / class exported by some dependency, then you can put an upper limit on the minor version. Harsh, I know, but orphans muck things up any way you slice it.

This means that any orphans you use, you'd need to explicitly depend on an A.B.C upper-bounded version of the packages that define the datatypes (and classes, if you're being entirely correct), and not consume them through something that re-exports them. Ideally, this could be mechanically checked.

This seems sensible. However, in case this wasn't already clear, it means when you /add/ an orphan instance, you still need a major version bump: suppose I depend on package A and package B and have an orphan for a type in package B, then you suggest I depend on package-B-minor and package-A-major – so if package A adds an orphan instance, they need to make their version bump major to stop me from breaking.

The maintenance of this kind of stuff can be reduced by centralizing stuff in orphans package (does Hackage 2.0 provide a way to email-subscribe to a package? might also be good for maintaining these sorts of things). If a central orphans package isn't acceptable, then you're probably not writing a library for general consumption anyway, so upper limits on minor bounds wouldn't be problematic.

Whether or not centralised orphans are appropriate really depends on why the orphans exist in the first place. If they are just because the original package deemed those instances not necessary, then fine (although they should really be put in the original package). But frequently they are because there is a class in module A and a type in module B that have a sensible instance, but neither the package containing A nor B know about each other (or there is no natural direction for the dependency to go). In that case, you really want a package per A,B pair, since otherwise you're going to bring in irrelevant dependencies.

-Michael

On Thu, Sep 6, 2012 at 4:24 AM, Simon Marlow wrote:

...

On 06/09/2012 01:16, Johan Tibell wrote:

...

Hi all,

The PVP says:

"A.B is known as the major version number, and C the minor version number. When a package is updated, the following rules govern how the version number must change relative to the previous version:

1. If any entity was removed, or the types of any entities or the definitions of datatypes or classes were changed, or instances were added or removed, then the new A.B must be greater than the previous A.B. Note that modifying imports or depending on a newer version of another package may cause extra instances to be exported and thus force a major version change."

The part about adding instances and the one about modifying imports makes it hard to follow the PVP. Bumping the major version number is a quite disruptive change for your users if they use upper bounds on their dependencies. Minor version bumps are not nearly as disruptive as you can you can depend on x.y.* as long as you only use qualified imports and/or explicit import lists.

Assuming no one uses orphan instances, adding instances is always safe because you can only add instances for types from packages you depend on, which means that they can't be depending on your package in turn and have defined a non-orphan instance for a type or class defined in your package.

I suggest that the rule be changed to not require a major version bump if instances are added.

Yes, I think this is reasonable. The client already has some obligations if they want to be independent of minor versions: they have to use explicit import lists. So adding another obligation, no orphan instances, is consistent with this and shouldn't cause problems in the majority of cases.

Cheers, Simon

...

P.S. I believe the same reasoning can be applied to the import part of the rule above.

-- Johan

_______________________________________________ Libraries mailing list Libraries@haskell.org http://www.haskell.org/mailman/listinfo/libraries

_______________________________________________ Libraries mailing list Libraries@haskell.org http://www.haskell.org/mailman/listinfo/libraries

_______________________________________________ Libraries mailing list Libraries@haskell.org http://www.haskell.org/mailman/listinfo/libraries