> I think we should apply the patch with LC_COLLATE.

There was a $$a in the awk script which I didn't understand anymore. I
guess that should rather have been a $$0.

I've removed the first sort and replaced sort+awk by something
completely in awk. So the selection of lines does not depend on any
locale anymore.

Whether the second sort is with LC_COLLATE=C or something else makes on
my computer only a difference in the two lines

DERHAM DeRhamComplex
DECIMAL DecimalExpansion

To be honest, I'd like DeRham be sorted after Decimal, so I left out
LC_COLLATE=C in the patch.

There is however still room for improvement, since .pamphlet files could
contain two )abbrev lines with the same short name and different long
name. If one of those lines appear in a chunk that is not used for the
actual .spad file, then this might lead to a problem.

Better were perhaps to run over the generated .spad files, but I didn't
want to let src/doc have any knowledge about what happens outside of
that directory.

Patch is here:
https://github.com/hemmecke/fricas/commit/e185878f47958d5621402c594621b25cff0f0a70

Does someone still see problems with it or may I commit?

Ralf

-- 
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 post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/fricas-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to