darcs

Patch 361 Relax the hashed-storage dependency (to >= 0.5.2 && < ...

Title Relax the hashed-storage dependency (to >= 0.5.2 && < ...
Superseder Nosy List mornfall
Related Issues
Status accepted Assigned To
Milestone

Created on 2010-08-24.08:45:28 by mornfall, last changed 2011-05-10.21:06:11 by darcswatch. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
relax-the-hashed_storage-dependency-_to-__-0_5_2-__-_-0_6__.dpatch mornfall, 2010-08-24.08:45:27 text/x-darcs-patch
unnamed mornfall, 2010-08-24.08:45:28
See mailing list archives for discussion on individual patches.
Messages
msg12280 (view) Author: mornfall Date: 2010-08-24.08:45:28
Hi,

this change is required for darcs-fastconvert to be buildable from Hackage (and
it needs to be included in 2.5, otherwise darcs-fastconvert will stay
unpublishable till another darcs release).

The changes that will go into 0.5.3 are available in the normal h-s repo at
http://repos.mornfall.net/hashed-storage -- I will publish a separate branch
for the 0.6 work later.

Yours,
   Petr.

1 patch for repository http://darcs.net:

Tue Aug 24 10:16:30 CEST 2010  Petr Rockai <me@mornfall.net>
  * Relax the hashed-storage dependency (to >= 0.5.2 && < 0.6).
Attachments
msg12291 (view) Author: kowey Date: 2010-08-25.14:29:24
On Tue, Aug 24, 2010 at 08:45:28 +0000, Petr Ročkai wrote:
> this change is required for darcs-fastconvert to be buildable from Hackage (and
> it needs to be included in 2.5, otherwise darcs-fastconvert will stay
> unpublishable till another darcs release).

Just checking: is this patch functionally equivalent to patch346?
(it doesn't really matter which gets applied, just thought it was
funny to see this come up a second time for a totally unrelated
reason).

It looks like this patch is blocking on consensus on the relaxing
the lockstep policy documented in
http://wiki.darcs.net/Development/Policy#dependencies 
but which poses some technical difficulties pointed out in

* http://bugs.darcs.net/patch346 and
* http://lists.osuosl.org/pipermail/darcs-users/2010-August/024995.html

I'd like a rough consensus on this, but (Reinier and Petr) if this is
really blocking urgently you have my green light.  What I figure is that
the new clever darcs --exact-version (which tells what versions of
libraries it was built on) is sufficient to avoid confusion.  We'd still
face the downside of increase the variability of darcsen in the wild,
but perhaps that's an acceptable tradeoff?

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
For a faster response, try +44 (0)1273 64 2905 or
xmpp:kowey@jabber.fr (Jabber or Google Talk only)
msg12346 (view) Author: tux_rocker Date: 2010-08-27.20:35:42
Hi,

2010/8/25 Eric Kow <kowey@darcs.net>:
> I'd like a rough consensus on this, but (Reinier and Petr) if this is
> really blocking urgently you have my green light.

This is not a review, but I am fine with applying such a patch to the
2.5 branch. The patch does not even actually change darcs if it only
does what it says it does.

Reinier
msg12347 (view) Author: kowey Date: 2010-08-27.21:16:18
This is going in along with other patches... last chance to object! (we
can always roll this back)
msg12360 (view) Author: darcswatch Date: 2010-08-28.20:02:18
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-d9717b7637093a6ca999dd474d646f537661a9ea
msg14313 (view) Author: darcswatch Date: 2011-05-10.21:06:11
This patch bundle (with 1 patches) was just applied to the repository http://darcs.net/reviewed.
This message was brought to you by DarcsWatch
http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-d9717b7637093a6ca999dd474d646f537661a9ea
History
Date User Action Args
2010-08-24 08:45:28mornfallcreate
2010-08-24 08:46:28darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-d9717b7637093a6ca999dd474d646f537661a9ea
2010-08-25 14:29:24koweysetmessages: + msg12291
2010-08-27 20:35:42tux_rockersetmessages: + msg12346
2010-08-27 21:16:18koweysetmessages: + msg12347
2010-08-28 20:02:18darcswatchsetstatus: needs-review -> accepted
messages: + msg12360
2011-05-10 21:06:11darcswatchsetmessages: + msg14313