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

LICENSE texts are are not correctly formatted #5552

Closed
1 of 3 tasks
tastytea opened this issue Dec 16, 2018 · 6 comments
Closed
1 of 3 tasks

LICENSE texts are are not correctly formatted #5552

tastytea opened this issue Dec 16, 2018 · 6 comments

Comments

@tastytea
Copy link

  • Gitea version (or commit ref): 1.6.1
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant

Description

When I add a new repository and select a license, the LICENSE file is missing space and double-newlines, at least for GPL-2.0-only, GPL-3.0-only, GPL-3.0-or-later and BSD-4-Clause.

The GPL-3.0 licenses are also having a corrupted URL in line 3.

Correctly formatted GPL-3.0 license: https://www.gnu.org/licenses/gpl-3.0.txt

Screenshots

screenshot_2018-12-16t16 50 52

@zeripath
Copy link
Contributor

Hmm... I'm not sure that the formatting is necessarily incorrect. All of the other licenses in options/license are similarly formatted. I'll put a PR in to fix the urls though.

@tastytea
Copy link
Author

It may not be technically incorrect, but it is hard to read for humans and I don't see any advantage.

@techknowlogick
Copy link
Member

@zeripath this is the script that pulls the latest license files from the SPDX repo:

func main() {

@zeripath
Copy link
Contributor

Heya @tastytea , so the issue with the url is coming from our upstream ( https://github.com/spdx/license-list-data ) - please could you open an issue there?

They've changed their formatting to match your preferred style - which I agree is nicer - so I've run our update script (go run scripts/generate-licenses.go) and added a PR for this #5558.

@tastytea
Copy link
Author

I've reported the issue upstream.

@techknowlogick
Copy link
Member

Closing this issue as PR has been merged.

@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants