-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
There shall be only one! #31
Comments
I partially agree, however @m2ym doesnt seem willing it. |
to do this, I might have to send a pullreq to the projects using optima to migrate to trivia. |
|
a kind of experiment. how the authors react? |
it looks like it was a haste move. it has some error on clisp. |
I should at least wait for the next release. |
I don't have much time to work on CL recently, and I am willing to integrate optima and trivia. What I can do, for example, is
@guicho271828 How do you think if we make an organization and a repository like "optima/optima" or "trivia/trivia" (please choose which you prefer) and work on the integration, as you are an author and a core developer? |
Wow. Congratulations to the two of you for advancing CL pattern-matching! @m2ym congratulations for having created optima with such talent, then for retiring it with such grace after it did its thing. @guicho271828 congratulations for furthering the state of CL pattern-matching. Regarding names: do we want either trivia or optima to be the final name? Wouldn't, e.g. Whichever the name ends up being, it could be the name of the package, with |
If I merge them seriously, I first have to fix the problem of trivia on clisp and abcl. I havent seriously treated those implementations other than sbcl and ccl. |
@fare If we don't concern about name collision, @guicho271828 As I remember, optima is tested on sbcl, ccl, acl, abcl, clisp and ecl. I don't have much care about breaking backward compatibility on minor implementations, but I think It will be great if trivia can work on such implementations. |
I spent a day for fixing bugs related to those obscure implementations. now it works. |
@guicho271828 Great! As the first step, I will put a link to trivia on optima's README. |
@guicho271828 what is the status of trivia with respect to readiness as a replacement for optima? Do you have parity with all documented optima features? |
|
Congrats for the successes. I don't know if any current code uses it, but I remember using |
the odd thing re: |
Right, as long as the feature is available and documented that's fine. |
Any news on this ? |
I don't know what is necessary on this... |
What are the options for merging the two regarding this? |
the internal designs are very different between the two and I don't think marching is possible. I don't think that infix when is an absolutely necessary feature either. Furthermore, the original author of Optima is no longer active in the common lisp community. Optima is not broken but is sort of in the dead end. |
Then maybe it's best to let it be and have 'optima' and 'trivia', separately. Projects that want to switch from optima to trivia can do so any time. |
It is already so for a long time. I don't think anyone new will use optima now |
The problem is that there are lots of libraries that still use it, it's unmaintained and has lots of open bugs. It depends on eos which is also deprecated and doesn't compile on SBCL HEAD, and if I wanted to send a PR to switch Optima to Fiveam there would be nobody to merge it. |
This is worse than when I did that in 2016 above. For the migration to happen, someone needs to send patches to all QL project users. And that's not me, I'm not an active CLer anymore. |
btw:
|
So the imcompatibility is mainly due to this? |
I realized that, apart from the compatibility package fare-quasiquote-optima, three of the legacy optima-using contenders were mine: exscribe, inferior-shell, quux-hunchentoot. I did my part to make them use trivia instead, and pushed the results on github, but the real upstream is gitlab.common-lisp.net, to which I seem to have lost access recently with the latest g.c-l.n update. 3 down, a few to go. |
@vindarel that is not quite accurate. |
yes and I dont think it is one of the core features, hopefully no one is using it... |
@guicho271828 Xach said he could remove Optima from Quicklisp if Trivia could provide an optima.asd which is backwards-compatible. Then we can deal with the libraries that use |
@sionescu will consider that this weekend |
Did my share for the dependencies. Another 3 down. None were breaking due to the change (tested locally). guicho271828/cl-rrt@00c7ab2 #115 provides |
Among the libraries listed in 99c553c , these libraries failed. I believe some are related to fare-quasiquote:
|
(note: test script in https://github.com/guicho271828/trivia/blob/optima-compat/optima-compat/test.lisp ) So I guess @fare have the one last nail to drive into it |
From optima's documentation,
In trivia this would be just parsed as |
in the libraries tested above, I did not see the warnings. Therefore, |
With this commit fare/fare-quasiquote#1 which redirects
|
Note that there is already a |
Some library may be depending on cl-erlang-optima, but if there are none it can be removed. Is it your library? |
I forgot that it was me who implemented the trivia port... |
The author of https://github.com/flambard/cl-erlang-term is @flambard. |
pointing @quicklisp here (hope this notifies him) |
Is there any reason not to just replace optima with trivia, then? (Assuming the optima author is willing.)
The text was updated successfully, but these errors were encountered: