darcs

Patch 346 Provide a more reasonable upper bound on hashed-storage version

Title Provide a more reasonable upper bound on hashed-storage version
Superseder Nosy List dagit, ganesh, kowey
Related Issues
Status rejected Assigned To
Milestone

Created on 2010-08-12.22:49:50 by dagit, last changed 2011-05-10.20:36:25 by darcswatch. Tracked on DarcsWatch.

Files
File name Status Uploaded Type Edit Remove
darcs_cabal_-provide-reasonable-upper-bound-on-h_s.dpatch dagit, 2010-08-12.22:49:50 application/octet-stream
unnamed dagit, 2010-08-12.22:49:50 text/html
See mailing list archives for discussion on individual patches.
Messages
msg12147 (view) Author: dagit Date: 2010-08-12.22:49:50
Hello,

Don Stewart pointed out to me today that our hashed-storage constraint is
overly constrained.  It's good that we provide a lower bound that takes into
account the necessary bug fixes, but we are not taking into account bug fix
releases of hashed-storage that become available after a darcs release.

This patch fixes that by specifying an upper bound.  I'm also making the
assumption that hashed-storage will follow the PVP:
  http://www.haskell.org/haskellwiki/Package_versioning_policy

Thanks,
Jason
Attachments
msg12148 (view) Author: ganesh Date: 2010-08-12.23:32:14
On Thu, 12 Aug 2010, Jason Dagit wrote:

> Don Stewart pointed out to me today that our hashed-storage constraint is
> overly constrained.  It's good that we provide a lower bound that takes into
> account the necessary bug fixes, but we are not taking into account bug fix
> releases of hashed-storage that become available after a darcs release.
>
> This patch fixes that by specifying an upper bound.  I'm also making the
> assumption that hashed-storage will follow the PVP:
>  http://www.haskell.org/haskellwiki/Package_versioning_policy

This is a deliberate policy (suggested by me, I think) to reduce confusion 
about whether a specific version of darcs would have a specific 
hashed-storage bug fix or not. Otherwise you have to identify both the 
darcs version and the h-s version in every bug report. In other words we 
specifically don't want to pick up bug fixes without revving darcs too.

Obviously the same argument might apply to many other packages we depend 
on, but h-s has a fairly unique position in that it's quite critical to 
darcs and also relatively fast-moving.

We can of course discuss the policy further, but I just wanted to be clear 
that it was a deliberate decision, rather than ignorance.

Cheers,

Ganesh
msg12150 (view) Author: kowey Date: 2010-08-13.08:24:10
On Fri, Aug 13, 2010 at 00:35:14 +0100, Ganesh Sittampalam wrote:
> This is a deliberate policy (suggested by me, I think) to reduce
> confusion about whether a specific version of darcs would have a
> specific hashed-storage bug fix or not. Otherwise you have to
> identify both the darcs version and the h-s version in every bug
> report. In other words we specifically don't want to pick up bug
> fixes without revving darcs too.

I regret that despite a bit of darcs changes searching and Googling, I
could not find an archived discussion on this.  But I guess now we have
such an archive for future use!

I seem to remember that the discussion took place in the Zurich
Darcs hacking sprint <http://wiki.darcs.net/Sprints/2010-03>
(I didn't see any mention there, except that hashed storage was
on the agenda for things to talk about)

-- 
Eric Kow <http://www.nltg.brighton.ac.uk/home/Eric.Kow>
For a faster response, please try +44 (0)1273 64 2905.
msg12241 (view) Author: dagit Date: 2010-08-19.23:35:06
I'm closing because no one else has yet (and I think it's rejected due to 
a policy decision?).

Jason
History
Date User Action Args
2010-08-12 22:49:50dagitcreate
2010-08-12 22:50:33darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-bb9438505151784f67b381fb6a0a301743308f91
2010-08-12 23:32:14ganeshsetnosy: + ganesh
messages: + msg12148
2010-08-13 08:24:10koweysetnosy: + kowey
messages: + msg12150
2010-08-19 23:35:06dagitsetstatus: needs-review -> rejected
messages: + msg12241
2011-05-10 20:36:25darcswatchsetdarcswatchurl: http://darcswatch.nomeata.de/repo_http:__darcs.net_.html#bundle-bb9438505151784f67b381fb6a0a301743308f91 -> http://darcswatch.nomeata.de/repo_http:__darcs.net_reviewed.html#bundle-bb9438505151784f67b381fb6a0a301743308f91