https://bugzilla.redhat.com/show_bug.cgi?id=2275318

Mikel Olasagasti Uranga mikel@olasagasti.info changed:

What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2275319

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2275319 [Bug 2275319] Review Request: transifex-client - The Transifex command-line client

https://bugzilla.redhat.com/show_bug.cgi?id=2275318

--- Comment #1 from Renich Bon Ciric renich@woralelandia.com --- Here're a few coments:

This package blocks 2275317 as well since it is a dependency of it.

It seems that table.txt is not documentation but a file needed to rebuild table.go. It should not be marked as documentation IMHO, what do you think?

## Add new characters

1. Edit `table.txt` file.

2. Rebuild `table.go` file:

   go run ./make_table.go
   

The license of the package seems to be the Apache License. IIUC, the appache indicates that a header be applied to every file as a comment:

APPENDIX: How to apply the Apache License to your work.

To apply the Apache License to your work, attach the following boilerplate notice, with the fields enclosed by brackets "{}" replaced with your own identifying information. (Don't include the brackets!) The text should be enclosed in the appropriate comment syntax for the file format. We also recommend that a file or class name and description of purpose be included on the same "printed page" as the copyright notice for easier identification within third-party archives.

Copyright {yyyy} {name of copyright owner}

Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. You may obtain a copy of the License at

... but none of the files do so. I don't know if this is OK but one should try and contact upstream and let them know about this, if you ask me. What do you think?

A hidden file has been included: /usr/share/gocode/src/github.com/gosimple/unidecode/.goipath

https://bugzilla.redhat.com/show_bug.cgi?id=2275318

Mikel Olasagasti Uranga mikel@olasagasti.info changed:

What |Removed |Added ---------------------------------------------------------------------------- Blocks| |2275317

--- Comment #2 from Mikel Olasagasti Uranga mikel@olasagasti.info --- Spec URL: https://mikel.olasagasti.info/tmp/fedora/golang-github-gosimple-unidecode.sp... SRPM URL: https://mikel.olasagasti.info/tmp/fedora/golang-github-gosimple-unidecode-1....

This package blocks 2275317 as well since it is a dependency of it.

done.

It seems that table.txt is not documentation but a file needed to rebuild table.go. It should not be marked as documentation IMHO, what do you think?

The idea of this package is a library to be used by other libraries or binaries, so if new chars should be added it would be needed to be done upstream. Nobody should use these libraries shipped by Fedora to develop, so it's just a upstream thing.

I don't know if this is OK but one should try and contact upstream and let them know about this, if you ask me. What do you think?

I'm afraid it's quite common in many Go projects to define the License at "root level" and not add copyright headers to code files. I'll try to contact upstream about it.

A hidden file has been included: /usr/share/gocode/src/github.com/gosimple/unidecode/.goipath

Happens with all Go packages.

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2275317 [Bug 2275317] Review Request: golang-github-gosimple-slug - URL-friendly slugify with multiple languages support

https://bugzilla.redhat.com/show_bug.cgi?id=2275318

Mikel Olasagasti Uranga mikel@olasagasti.info changed:

What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(renich@woraleland | |ia.com)

https://bugzilla.redhat.com/show_bug.cgi?id=2275318

Renich Bon Ciric renich@woralelandia.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? |fedora-review+

https://bugzilla.redhat.com/show_bug.cgi?id=2275318

Fedora Update System updates@fedoraproject.org changed:

What |Removed |Added ---------------------------------------------------------------------------- Status|RELEASE_PENDING |MODIFIED

--- Comment #6 from Fedora Update System updates@fedoraproject.org --- FEDORA-2024-1cfd7e2686 (golang-github-gosimple-unidecode-1.0.1-1.fc42) has been submitted as an update to Fedora 42. https://bodhi.fedoraproject.org/updates/FEDORA-2024-1cfd7e2686