What's important is to provide a painless for the user way to get the version, and possibly more (e.g. the underlying Lisp).
That's because what upstreams, such as SageMath, often need to know. I'd propose to add the version support via pkg-config. If this is OK, I'd provide a PR for this. pkg-config would output the version in an easily parceable format, so that one would not need to call sed to santitize the output of fricas --version Dima On Fri, 1 Sept 2023, 22:14 Grégory Vanuxem, <[email protected]> wrote: > Yes I agree. Not a necessary addition. > And to add, not a Git specialist. Too a lot of errors with me. But that's > the way. > > __ > Greg > > Le jeu. 31 août 2023, 13:09, Waldek Hebisch <[email protected]> > a écrit : > >> Dnia Tue, Aug 22, 2023 at 08:03:50AM +0200, Grégory Vanuxem napisał(a): >> > Hello, Ralf, Waldek, *, >> > >> > Thanks Ralf for supporting this proposal. >> > >> > I'm used to run different FriCAS versions at the same time and since I >> > plan to support this in a FriCAS personal plugin for VSCode (not >> > published yet), I need version informations to populate my >> > 'FriCASExecutable' computational "structure". For now 'fricas >> > --version' is inconsistent to me. But this is usual apparently even >> > with Common Lisp implementations. For example, 'gcl --version' gives: >> > >> > $gcl --version >> > GCL 2.6.14 ... >> > >> > whereas in GCL >> > >(lisp-implementation-type) >> > "GNU Common Lisp (GCL)" >> > >> > Clozure CL is more consistent from my point of view: >> > =E2=94=94=E2=94=80$ >> /home/greg/.roswell/impls/x86-64/linux/ccl-bin/1.12.2/l= >> > x86cl64 --version >> > Clozure Common Lisp Version 1.12.2 (v1.12.2) LinuxX8664 >> > >> > and in it: >> > CCL is free software. It is distributed under the terms of the Apache >> > Licence, Version 2.0. >> > ? (lisp-implementation-type) >> > "Clozure Common Lisp" >> > ? >> > >> > After, that is just a matter of opinion I guess. >> > By the way, if there is some interest I filed a pull request, >> > https://github.com/fricas/fricas/pull/134 >> > >> > The original code is in >> https://github.com/gvanuxem/fricas/tree/fricas-vers= >> > ion >> > (the fricas-version branch) for an eventual 'diff'. I will keep one >> > since I will delete this branch in the near future. >> >> Moving tests so that '--version' is handled before possible error >> messages is fine. However, other changes look like regression to >> me: current tests in 'config.lisp' deliberately match feature >> tests in Lisp code. Since "features" are used by programs they >> should be more stable than other things. Basing test on output >> that is normally _not_ handled by programs is less robust. >> And there is added uglyness due to embedded spaces in messages. >> >> -- >> Waldek Hebisch >> >> -- >> You received this message because you are subscribed to the Google Groups >> "FriCAS - computer algebra system" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected]. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/fricas-devel/ZPB02N9%2BKileYdrU%40mail.math.uni.wroc.pl >> . >> > -- > You received this message because you are subscribed to the Google Groups > "FriCAS - computer algebra system" group. > To unsubscribe from this group and stop receiving emails from it, send an > email to [email protected]. > To view this discussion on the web visit > https://groups.google.com/d/msgid/fricas-devel/CAHnU2daHSrLZY9Wz61r_XuM7%3DmJzRB_k8uo-u4HK5UUPvJvYOw%40mail.gmail.com > <https://groups.google.com/d/msgid/fricas-devel/CAHnU2daHSrLZY9Wz61r_XuM7%3DmJzRB_k8uo-u4HK5UUPvJvYOw%40mail.gmail.com?utm_medium=email&utm_source=footer> > . > -- You received this message because you are subscribed to the Google Groups "FriCAS - computer algebra system" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/fricas-devel/CAAWYfq2L8eQ_WG-gsDopSU%3DHY_kV-k%3DPMTZshoiibcXUq%2BuY3A%40mail.gmail.com.
