darcs

Patch 1490 remove support for DARCS_GET_ and DARCS_APPLY_ hacks

Title remove support for DARCS_GET_ and DARCS_APPLY_ hacks
Superseder Nosy List gh
Related Issues
Status accepted Assigned To
Milestone 2.12.0

Created on 2016-04-21.20:23:59 by gh, last changed 2016-04-25.04:25:35 by gh.

Files
File name Status Uploaded Type Edit Remove
patch-preview.txt gh, 2016-04-21.20:23:58 text/x-darcs-patch
patch-preview.txt gh, 2016-04-22.19:59:59 text/x-darcs-patch
remove-support-for-darcs_get_-and-darcs_apply_-hacks.dpatch gh, 2016-04-21.20:23:58 application/x-darcs-patch
remove-support-for-darcs_get_-and-darcs_apply_-hacks.dpatch gh, 2016-04-22.19:59:59 application/x-darcs-patch
unnamed gh, 2016-04-21.20:23:58 text/plain
unnamed gh, 2016-04-22.19:59:59 text/plain
See mailing list archives for discussion on individual patches.
Messages
msg19189 (view) Author: gh Date: 2016-04-21.20:23:58
Going though "darcs help environment", I read about DARCS_GET_FOO
and DARCS_APPLY_FOO, clearly documented as hacks.

This patch removes the feature of using these variables when cloning
from / applying to non-standard URL's.

Other VCS like mercurial and git only support http(s), ssh, and
ftp(s) url's just like Darcs does natively (Git also support
rsync:// natively).

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

patch 63f9b2117eba7c80f7be0efc9f377211887a10e1
Author: Guillaume Hoffmann <guillaumh@gmail.com>
Date:   Thu Apr 21 17:17:15 ART 2016
  * remove support for DARCS_GET_ and DARCS_APPLY_ hacks
Attachments
msg19190 (view) Author: gh Date: 2016-04-22.19:12:15
Relevant chat http://irclog.perlgeek.de/darcs/2016-04-21#i_12374921
msg19191 (view) Author: gh Date: 2016-04-22.19:59:59
There was not much left in UI.RemoteApply and this module
was only used by UI.Commands.Push since UI.Commands.Put
ceased to exist, so merging it seemed to make sense.

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

patch 63f9b2117eba7c80f7be0efc9f377211887a10e1
Author: Guillaume Hoffmann <guillaumh@gmail.com>
Date:   Thu Apr 21 17:17:15 ART 2016
  * remove support for DARCS_GET_ and DARCS_APPLY_ hacks

patch 508a8b6f027aca6b05740dcc7d4ab0a7516f6a41
Author: Guillaume Hoffmann <guillaumh@gmail.com>
Date:   Fri Apr 22 16:59:46 ART 2016
  * merge UI.RemoteApply into UI.Commands.Push
Attachments
msg19198 (view) Author: gh Date: 2016-04-25.04:25:34
Because of the coming 2.12 release, I'm going to bend the
one-week-self-accept rule into a
three-days-self-accept-moreover-Ganesh-did-not-say-yes-but-almost rule
(see IRC discussion for context). Hopefully having these patches in the
2.12 branch will give them more exposure and opportunity to get user
feedback early.

As mentioned in the IRC discussion above, we can reintroduce the feature
(or half of it) if there is a real need. (Which, IMO, and as far as a
simple internet search can tell, is quite unlikely.)

So, self-accept.
History
Date User Action Args
2016-04-21 20:23:59ghcreate
2016-04-22 19:12:15ghsetmessages: + msg19190
2016-04-22 19:59:59ghsetfiles: + patch-preview.txt, remove-support-for-darcs_get_-and-darcs_apply_-hacks.dpatch, unnamed
messages: + msg19191
2016-04-25 04:01:45ghsetstatus: needs-screening -> needs-review
2016-04-25 04:25:35ghsetstatus: needs-review -> accepted
messages: + msg19198
milestone: 2.12.0