darcs

Patch 1742 resolve issue2604: remove --reply and related options

Title resolve issue2604: remove --reply and related options
Superseder Nosy List bfrk, ganesh
Related Issues
Status accepted Assigned To ganesh
Milestone

Created on 2018-10-12.23:46:08 by bfrk, last changed 2018-11-17.15:45:50 by ganesh.

Files
File name Status Uploaded Type Edit Remove
patch-preview.txt bfrk, 2018-10-12.23:46:08 text/x-darcs-patch
resolve-issue2604_-remove-__reply-and-related-options.dpatch bfrk, 2018-10-12.23:46:08 application/x-darcs-patch
unnamed bfrk, 2018-10-12.23:46:08 text/plain
See mailing list archives for discussion on individual patches.
Messages
msg20400 (view) Author: bfrk Date: 2018-10-12.23:46:08
This is up for discussion, of course.

I claim that having this sort of functionality in a revision control system
is obsolete. It made sense back when darcs was new, as public repo hosting
services hadn't been invented yet, but nowadays it's just a relict and a web
service can do these things easier and more reliably.

This move gets us rid of a lot of code that is obscure, non-portable, and
hard to maintain.

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

patch d2e84e16dcbbdee97937581ea4af402f92921078
Author: Ben Franksen <ben.franksen@online.de>
Date:   Sat Oct 13 01:09:27 CEST 2018
  * resolve issue2604: remove --reply and related options
  
  The options are: ccApply, reply, happyForwarding and were supported by the
  apply and push commands. This patch removes all conditional compilation from
  Darcs.Util.Compat. It also removes the dependency on the random package when
  building for Windows.
Attachments
msg20477 (view) Author: ganesh Date: 2018-11-17.09:48:14
Fine (this has been discussed briefly on issue2604 and on the mailing 
list)
History
Date User Action Args
2018-10-12 23:46:09bfrkcreate
2018-11-07 21:21:48bfrksetstatus: needs-screening -> needs-review
2018-11-17 09:48:14ganeshsetstatus: needs-review -> accepted-pending-tests
assignedto: ganesh
messages: + msg20477
nosy: + ganesh
2018-11-17 15:45:50ganeshsetstatus: accepted-pending-tests -> accepted