Skip to content
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

organizing the repo #634

Open
Udi-Fogiel opened this issue Apr 19, 2024 · 15 comments
Open

organizing the repo #634

Udi-Fogiel opened this issue Apr 19, 2024 · 15 comments

Comments

@Udi-Fogiel
Copy link
Collaborator

Now that we have version tagging and date update, auto testing for regressions and building ctan archives using l3build, I think it will be a good time to clean the repository a bit.

Can we remove (or put aside) some of the unused scripts? Can we convert doc/TODO and doc/BUGS to tickets or regression tests? What about the misc directory, is it still needed?

@Udi-Fogiel
Copy link
Collaborator Author

In fact I believe the current situation is quite misleading. The .ldf files in the .tds.zip file are the one that are in the repo, and not the ones that generate from the .dtx file, although the .dtx file is in the source directory, so for example in MikTeX the .ldf files does not contain any of the license/copyright information, while if I will generate them from the "source" my self, they will.

Should they contain the comments that are add by the .dtx file?

@Udi-Fogiel
Copy link
Collaborator Author

Udi-Fogiel commented Apr 19, 2024

Also the files generated from the .dtx file are all licensed under LPPL while the repo state polyglossia is licensed under MIT...

@Udi-Fogiel Udi-Fogiel changed the title orgenizing the repo organizing the repo Apr 19, 2024
@Udi-Fogiel
Copy link
Collaborator Author

@jspitz do you know what ./tests/test-gloss-uyghur.tex tests? I'm not sure how to use it in the new test suit.

@jspitz
Copy link
Collaborator

jspitz commented Apr 22, 2024

@jspitz do you know what ./tests/test-gloss-uyghur.tex tests? I'm not sure how to use it in the new test suit.

AFAICS it tests some basic features with uyghur, including switch to LTR (english). Seems to be inspired from example-arabic.

BTW thanks for setting this suite up.

@Udi-Fogiel
Copy link
Collaborator Author

AFAICS it tests some basic features with uyghur, including switch to LTR (english). Seems to be inspired from example-arabic.

Then I'll probably move that file to the doc folder, with the rest of the examples.

BTW thanks for setting this suite up.

Sure, it was only fair after all the regressions I introduced :)

When you'll have time, can you look at doc/TODO and see what is still relevant, and maybe move whats relevant to tickets?

@jspitz
Copy link
Collaborator

jspitz commented Apr 26, 2024

I think TODO is all very old stuff. I would just remove it. Things that are still important today will be re-requested anyway.

@Udi-Fogiel
Copy link
Collaborator Author

what about gloss-occitan.dtx? are we using it?

@jspitz
Copy link
Collaborator

jspitz commented Apr 27, 2024

what about gloss-occitan.dtx? are we using it?

I think this is originally a separate package that has been included into polyglossia at some point (see also https://www.ctan.org/pkg/gloss-occitan).

I don't think we need this in our repository. @reutenauer?

Such external packages will break at some point with our rewrite (e.g., the keyval change). But I don't know why this package is not marked obsolete anyway.

@reutenauer
Copy link
Owner

reutenauer commented Apr 29, 2024 via email

@Udi-Fogiel
Copy link
Collaborator Author

I removed it at [a086aff]. What about branches? There are currently 37 of them, some does not seem the be ahead with any commits (probably merged), can I safely remove those?

@Udi-Fogiel
Copy link
Collaborator Author

What about all the lua module version? should we update it along the package version?

@Udi-Fogiel
Copy link
Collaborator Author

The remaining things are deciding about the lua module version, maybe port the scripts in ./tools to lua so that future development will have less dependencies (we will always have a lua interpreter...) and maybe moving uax9.sty elsewhere?

@jspitz
Copy link
Collaborator

jspitz commented Jul 17, 2024

What is the purpose of uax9.sty anyway?

@jspitz
Copy link
Collaborator

jspitz commented Jul 17, 2024

I mean, I understand what the package attempts do do, but this seems to be not released nor maintained, and it is also not included anywhere AFAICS.

@Udi-Fogiel
Copy link
Collaborator Author

The code looks rather similar to the one in https://github.com/khaledhosny/context-bidi. I've tried a couple of time to play with it, but even after making it work the result is not really adequate, maybe it is a good start though.

It would be nice to have a better support for bidi in LuaTeX, but maybe we can archive this file elsewhere. @reutenauer Do you have any opinion? I think you initially ported this code from conTeXt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants