Opened 14 years ago
Closed 14 years ago
#792 closed change request (fixed)
CR: deprecate "make" syntax from "setup-api" module
| Reported by: | felix winkelmann | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | core libraries | Version: | 4.7.x |
| Keywords: | make setup-api | Cc: | |
| Estimated difficulty: |
Description
The functionality of this syntax is already available with the make egg and is not used inside the core system. Therefore it should be deprecated. A patch to do so is attached.
Attachments (1)
Change History (10)
Changed 14 years ago by
| Attachment: | 0001-deprecate-make-syntax.patch added |
|---|
comment:1 follow-up: 3 Changed 14 years ago by
comment:2 Changed 14 years ago by
With my small shell-fu the following eggs would be affected:
find ./release/4 -name \*.setup -exec grep -H make\ {} \; |grep trunk | cut -f1 -d: |sort -u | while read f; do basename $f; done
9ML-toolkit.setup
bind.setup
crunch.setup
dbus.setup
dissector.setup
easyffi.setup
flsim.setup
hyde.setup
iexpr.setup
jbogenturfahi.setup
kanren.setup
loop.setup
make.setup
miniML.setup
nemo.setup
objc.setup
openal.setup
opengl.setup
pulseaudio.setup
qt.setup
qwiki.setup
scss.setup
signal-diagram.setup
spiffy-uri-match.setup
spock.setup
stalin.setup
static-modules.setup
sxml-informal.setup
utf8.setup
vandusen.setup
zmq.setup
That's 31 a managable small number. So for the sake of a leaner core system I am for it.
comment:3 Changed 14 years ago by
Replying to syn:
Shouldn't there be a poll? How do I add one? :-)
The syntax is something like [[Poll(OMG R U OK bro?; Yes; No)]]
comment:4 Changed 14 years ago by
Since only a few eggs depend on make and adapting them to use an egg is trivial, I'm for deprecating (then removing) core's make.
comment:8 Changed 14 years ago by
Replying to felix:
Ok - the patch will be posted on chicken-hackers.
I must have missed it somehow...
comment:9 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
It was posted here. This ticket can be closed.

Shouldn't there be a poll? How do I add one? :-)