oxigraph/oxilangtag

Missing attribution

pyfisch opened this issue · 4 comments

Hello @Tpt,
your comment brought to my attention that you publish your own language tags crate and use some of the code from my original crate. That's fine, I like it when code that I've written is used in other projects. I especially appreciate it that you contributed a rewrite of the language-tags crate.

What I don't like is that when you published your own crate you copied parts of the language-tags crate without attributing the authors and preserving the git history of the code. One example that is the parse_langtag. Please restore the git history and mention me and the other authors as required by the - already very permissive - MIT license.

Tpt commented

Hi @pyfich.

I thought I have used in this crate only code from my rewrite as you said and so I did not needed to credit you. But, after checking carefully, I have indeed reused some of your code like the error struct. I deeply apologize for that. I have rewritten the git history, hopefully properly credited you and yanked the older releases. There is no fork of this repository so the bad versions should be gone for good. I deeply apologize for this mistake. Do you want to be credited also in the Cargo.toml?
Feel free to ask if I still made mistakes.

If you want, I can also submit pull request to your crates in order to delete this fork, I mostly did it because I feared you completely dropped from maintaining the language-tags crate.

Sorry gain.

Thank you for reacting so quickly! I don' think it would be appropriate to mention me in the Cargo.toml, considering I am not maintaining this crate and there is comparatively little code written by me.

If you want, I can also submit pull request to your crates in order to delete this fork, I mostly did it because I feared you completely dropped from maintaining the language-tags crate.

I have been maintaining the language-tags crate passively: I would react to PRs and issues but didn't write much code myself. It usually helps if I am pinged, if I forget to do something. Sorry for giving you the impression that I dropped maintaining the crate.

I think it is beneficial to the rust ecosystem if there is just one crate to provide some basic functionality like language tags. Additionally it is always preferable to have a truck number larger than one for a crate. 😉 For these reasons I would support merging the oxilangtag and language-tags crates. If you maintain a language tags crate, I will gladly do code reviews and help with issues and releases. This crate can be part of oxigraph org.

Do you know which major differences there are between oxilangtag and language-tags right now?

Tpt commented

Sorry for the very late answer.

It's great to ear that you still maintain the language tags crate.

+1 to having a single crate if it allows to provide enough for the different needs and thank you so much for the very nice and kind proposal.

oxilangtag is very focus on the Oxigraph/RDF use case: being able to validate is a language tag is well-formed. There is no canonicalization, case normalization... So it's feature scope is pretty closed and I basically have not needed to update it after its first release. Outside of the "missing" features the main difference is that oxilangtag allows zero copy validation and to choose which buffer to use internally.

The oxigraph org is very focused on Oxigraph itself so it might not be a good place for a general use language crate. Your crate is much more used than oxilangtag so it might make sense to keep your crate where it is and merge oxilangtag in it if its specific features (zero-copy validation...) do not conflict with language-tags design to much.

Sorry for the very late answer.

Sorry for my late answer as well.

Yeah, in this case its better to keep the crates separate. However I think rust-language-tags should link oxilangtag and describe when it is a more lightweight alternative.