[CONTRIB-93] remove deprecations on master Created: 24/Aug/10 Updated: 03/Sep/10 Resolved: 03/Sep/10 |
|
| Status: | Closed |
| Project: | Clojure-Contrib |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Defect | ||
| Reporter: | Anonymous | Assignee: | bpsm |
| Resolution: | Completed | Votes: | 0 |
| Labels: | None | ||
| Description |
|
I noticed the recent aborted attempt to remove the deprecated modules form clojure-contrib master. Since I was in need of some busy work, I decided to try and do it myself and clean up the breakage. The attached patches don't remove everything that has been marked as deprecated, just the bits Stuart previously indicated by trying to remove them.
|
| Comments |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
Converted from http://www.assembla.com/spaces/clojure/tickets/93 |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:ahtcAoR60r34W9eJe5cbCb] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: I'll submit further patches as I have time. Not being "in the loop", I can't be certain that we really want all the namespaces marked as deprecated gone before 1.3, but that's my operating assumption. (I'm attaching one patch per namespace so that the comitters can be selective.) |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:bR5RKOSu0r36XVeJe5cbLA] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:dxyjsaSHar37S6eJe5cbCb]: proposed new c.c.io providing only what was not migrated to Clojure proper |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: Proposal: when a namespace is partially promoted to Clojure proper, it might be good to provide a reduced version of the old namespace, providing just the functionality that was not promoted as an alternative to complete removal. I've sketched out this idea as a patch: 0010-create-clojure.contrib.io-convenience-module. I'm imagining something like:
This new clojure.contrib.io is not a strict subset of the old clojure.contrib.io, as some changes seemed sensible to harmonize with clojure.java.io's conventions. This means, it's not a drop-in replacement, but at least provides equivalents for anyone trying to get off the deprecated clojure.contrib.io. One problem that's apparent with this proposal is that, since the 'new' io only becomes available in 1.3, there would be something of a "flag day" when switching from 1.2 to 1.3, but then there would be if 1.3 completely eliminated c.c.io as well – only it would be worse. |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
stuart.sierra said: clojure.contrib.io is deprecated, period. It will be removed in the next release. |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: Fine, so ignore patch 0010. That's why it's a "proposal". Those moving their code off of c.c.io to c.j.io will just have to write their own versions of a subset of {file-str, write-lines, read-lines, pwd, with-out, with-in, to-byte-array, relative-path-string and delete-file-recursively}, or get these from a library outside of clojure-contrib. Am I understanding correctly? Patch 0010 contains (in the comments) some tips for porting from deprecated c.c.io to c.j.io. It would make sense to gather that kind of information in some kind of a guide/faq about how to replace uses of deprecated c.c. namespaces promotion to core was not 1-to-1. |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: ���replace uses of deprecated c.c. namespaces where promotion to core was not 1-to-1. |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:dEtFMSSRir35x5eJe5cbCb] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:dSzxyYSRir35nTeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:dSD8sqSRir35nTeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:cJIVlaSTur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:cJNkS8STur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:cJQGkcSTur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:cJ5Kd4STur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:cJ8-lqSTur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:daZ0rSSTur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: [file:da5unsSTur34OJeJe5cbLr] |
| Comment by Assembla Importer [ 03/Sep/10 8:52 PM ] |
|
bpsm said: This task looks done to me. I can nothing further in clojure-contrib that's been deprecated. I've placed these patches on a branch on github for easy browsing. |