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

There's still a certain amount of unneeded files in the fez distro #44

Open
JJ opened this issue Oct 24, 2023 · 4 comments
Open

There's still a certain amount of unneeded files in the fez distro #44

JJ opened this issue Oct 24, 2023 · 4 comments
Milestone

Comments

@JJ
Copy link
Collaborator

JJ commented Oct 24, 2023

It still weighs 100Kb

This is what is found there:

Captura de pantalla de 2023-10-24 12-16-51

Also, this:

Captura de pantalla de 2023-10-24 12-18-56

I can't for the life of me figure this last one out. Might be some mac stuff, but who knows...

@Tux
Copy link
Owner

Tux commented Oct 24, 2023

CSV.rakumod is the module itself. The Text::CSV source is 2056 lines
The top-level CSV.rakumod and CSV.pod6 are just symlinks to lib/Text/CSV.rakumod and lib/Text/CSV.pod6 so I can edit easy. The documentation is currently 3084 lines, which is 80 kb.
I'm not willing to trim down the documentation just to shave off a few kb

@JJ
Copy link
Collaborator Author

JJ commented Oct 24, 2023

CSV.rakumod is the module itself. The Text::CSV source is 2056 lines The top-level CSV.rakumod and CSV.pod6 are just symlinks to lib/Text/CSV.rakumod and lib/Text/CSV.pod6 so I can edit easy. The documentation is currently 3084 lines, which is 80 kb. I'm not willing to trim down the documentation just to shave off a few kb

Only it appears twice in the package within the lib directory. I'm not saying that you should trim down the documentation, not by a long shot. This is mostly a fez bundling issue, but one we'll have to deal with while it's not fixed at the other end.

@JJ
Copy link
Collaborator Author

JJ commented Oct 28, 2023

Actually, I haven't been able to deal with the two symbolic links. Will open a different issue for that. Problem is that if we .gitignore them, there's no easy way to separate the symbolic link from the real deal.
Other that that, @Tux I would say we're ready for the new version, all issues in the milestone cleared.

@JJ
Copy link
Collaborator Author

JJ commented Oct 30, 2023

Still going
Captura de pantalla de 2023-10-30 09-55-02
Not as hefty now, but still unneeded.

@JJ JJ reopened this Oct 30, 2023
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

2 participants