darcs

Patch 906 rm compression flags when possible since... (and 4 more)

Title rm compression flags when possible since... (and 4 more)
Superseder Nosy List gh
Related Issues
Status accepted Assigned To
Milestone

Created on 2012-08-19.00:45:57 by gh, last changed 2012-09-29.21:04:28 by owst.

Files
File name Status Uploaded Type Edit Remove
no-longer-show-repo-type-since-it-is-always-darcs.dpatch gh, 2012-08-21.17:13:24 application/x-darcs-patch
patch-preview.txt gh, 2012-08-19.00:45:56 text/x-darcs-patch
patch-preview.txt gh, 2012-08-21.17:13:24 text/x-darcs-patch
rm-compression-flags-when-possible-since-pristine-files-are-always-compressed.dpatch gh, 2012-08-19.00:45:56 application/x-darcs-patch
unnamed gh, 2012-08-19.00:45:56
unnamed gh, 2012-08-21.17:13:24
See mailing list archives for discussion on individual patches.
Messages
msg15988 (view) Author: gh Date: 2012-08-19.00:45:56
The first two patches are followups to the refactoring that replaced
UI flags by Repository flags and passed them explicitely to functions
of Darcs.Repository.*. The patches do not change the behaviour of darcs since
it is impossible to pass the "compression" flag to commands like get,
repair, dist, annotate. On a related note, it is never possible to
have uncompressed pristine.

The third patch removes an abstraction that has been useless since
darcs stopped handling Git repositories internally: the datatype
Repository (from Darcs.Repository.InternalTypes) contained a field
of type RepoType, which had only one constructor (now), DarcsRepository.
The patch removes RepoType and copies the fields of DarcsRepository
to Repository.

The last two patches are just cleanup.

5 patches for repository http://darcs.net:

Thu Aug 16 18:28:17 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * rm compression flags when possible since pristine files are always compressed

Sat Aug 18 13:28:54 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * darcs repair does not use compress flags, take advantage of this

Sat Aug 18 13:28:59 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove now useless nesting of RepoType into Repository

Sat Aug 18 13:42:23 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * make that no Darcs.Repository.* module import Darcs.Repository

Sat Aug 18 13:52:12 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * remove senseless comment
Attachments
msg15992 (view) Author: gh Date: 2012-08-21.17:13:24
A followup for the "darcs show repo" command.

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

Tue Aug 21 14:07:57 ART 2012  Guillaume Hoffmann <guillaumh@gmail.com>
  * no longer show repo type since it is always darcs
Attachments
msg16154 (view) Author: owst Date: 2012-09-29.21:04:28
Looks good, accepting, thanks.
History
Date User Action Args
2012-08-19 00:45:57ghcreate
2012-08-21 17:13:24ghsetfiles: + patch-preview.txt, no-longer-show-repo-type-since-it-is-always-darcs.dpatch, unnamed
messages: + msg15992
2012-08-21 17:14:01ghsetstatus: needs-screening -> needs-review
2012-09-29 21:04:28owstsetstatus: needs-review -> accepted
messages: + msg16154