Cwa-app-ios: License is hard to find

Created on 27 Dec 2020  路  13Comments  路  Source: corona-warn-app/cwa-app-ios

Avoid duplicates

  • [x] This issue has not already been raised before
  • [x] If you are proposing a new feature, please do so in CWA-Wishlist

Current Implementation

iOS Repository does not have a license.

Suggested Enhancement

Add a license, i.e. the Apache License Version 2.0, how it's already done in the Android Repository.

I would open a PR which adds the license, you only have to check if this Repository is also licensed under the Apache License Version 2.0.


Internal Tracking ID: EXPOSUREAPP-4470

Fix 1.11 enhancement mirrored-to-jira

All 13 comments

Morning @Ein-Tim,

Have a look here: https://github.com/corona-warn-app/cwa-app-ios/blob/release/1.9.x/LICENSES/Apache-2.0.txt.

Best, DS


Corona-Warn-App Open Source Team

Okay thanks @dsarkar, I did not see this 馃槄
Maybe you would like to add this text to the README.md here to?
I have a PR implementing this ready, so just lemme know 馃憤
Because as of now the license is quite hard to find and GitHub is also showing:
image

Solved by #1754

@Ein-Tim Thanks.

Here isn't the license still
image

@jucktnich yes this is not possible because GitHub searches for a file named "LICENSE" and grabs this but the iOS Repo has the licenses in a folder (GitHub is not aware of this).

Why is it not possible to create a license file?

@jucktnich

As you can see in the LICENSES folder, the Repo is licensed under 2 Licenses, I think GitHub can only show one license, so you would have to pick one.

Or write two in one file.

Yes would be possible, maybe @kaddaSz could just quickly give her opinion to this? 馃檪

@jucktnich, @Ein-Tim:
The LICENSES folder is needed for the REUSE Tool, to which we migrated for the iOS repo already. Unfortunately this is not recognized by GitHub.
We will check if we just add the LICENSE file with the Apache 2.0 License again (and additionally) to the root folder, so that the license information can be shown by GitHub.

@jucktnich see #1791

@dsarkar
Can be moved to done in [CM] cwa-app-ios.
Thanks 馃憤

Was this page helpful?
0 / 5 - 0 ratings