[packaging juju-1.23] Issues met while working on debian/copyright file

Bug #1442132 reported by Oleg Strikov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju-core
Fix Released
High
Martin Packman
1.23
Fix Released
High
Martin Packman

Bug Description

Hi Juju Team,

I started to work on debian/copyright file before the official release of 1.23 to decrease the amount of time needed for packaging. I don't expect that the list of dependencies changes seriously from -beta3 to release.

You may see that we have much less issues this cycle which means that we did a very good job last cycle!

(0) [gz:TESTSUITE CLEANER EXPANED TO WIPE NEW LOCATION] [gz:GOLANG BUG FILED]
juju-core_1.23-beta3/src/golang.org/x/net/html/charset/testdata/*.html
Golang guys picked these files from W3C i18n testsuite. At that time this testsuite was licensed under non-modifiable license (LP: #1341589) and we had to manually remove these files from the package. We removed these files from src/code.google.com/p/go.net/ folder but didn't do that for the new src/golang.org/x/net folder. We may want to remove these tests from both locations in the package. See [A] for additional thoughts on this.

(1)
https://github.com/juju/charm/{v4, v5}
License for this project has been changed from AGPLv3 to LGPLv3 with linking exceptions during the previous cycle. Unfortunately virtually all files in the project contain references to AGPLv3 in their headers.

(2)
https://github.com/juju/cmd/blob/master/supercommand_test.go#L2
Header needs to be fixed by changing AGPLv3 to LGPLv3 to conform with the LICENSE file and other files in the project.

(3)
https://github.com/juju/juju/blob/master/apiserver/http/package_test.go
https://github.com/juju/juju/blob/master/wrench/export_test.go
https://github.com/juju/juju/blob/master/utils/logging_test.go
https://github.com/juju/juju/blob/master/utils/logging.go
Headers need to be fixed by changing LGPLv3 to AGPLv3 to conform with the LICENSE file and other files in the project.

(4)
https://github.com/juju/testing/blob/master/stub.go
https://github.com/juju/testing/blob/master/stub_test.go
Headers need to be fixed by changing AGPLv3 to LGPLv3 to conform with the LICENSE file and other files in the project.

=== Minor issues which might be not issues at all ===

(5) [RESOLVED]
https://github.com/juju/juju/blob/master/scripts/win-installer/LICENCE.txt
Do we really need to have AGPLv3 LICENSE file in this folder when we have project-wide AGPLv3 LICENSE file?

This LICENSE file is included into the windows installer.
No change is needed.

(6)
Tarball for juju-1.23-beta3 contains empty folder src/github.com/kisielk/

(7)
Juju-1.23-beta3 has both 'golang.org/x/net' and 'code.google.com/p/go.net/' as dependencies. They look very similar to me maybe even different version of the same code base. I don't know if it's okay or not, just want to make sure that you're aware of it.

Thanks for helping!

[A]
Another way exists as well. W3C now distributes this testsuite under dual-license one of which is BSD (http://www.w3.org/Consortium/Legal/2008/04-testsuite-copyright.html). It means that we can include these tests back to the distribution but only if license file and required copyright notices will be available in the source tree. Golang guys don't have them in the repo and mgz will try to fix this bug upstream. When done, we can return these test back to our package.

Related branches

Revision history for this message
Curtis Hovey (sinzui) wrote :

I expect this issue will start work in 1.23-beta4 but will really be fixed in a future release of 1.23.

Changed in juju-core:
status: New → Triaged
importance: Undecided → High
milestone: none → 1.24-alpha1
Revision history for this message
Martin Packman (gz) wrote :

We're just discussing golang.org/x/net and code.google.com/p/go.net in #ubuntu-server currently. Summary, as code.google.com is going away, various projects are moving location, but for this release not all import references were updated yet it seems.

Revision history for this message
Oleg Strikov (strikov-deactivatedaccount) wrote :

I just updated bug description to include the point (0) which addresses this testsuite-related issue. I *think* that we may want to remove this testsuite from the second (golang.org/x/net) location for 1.23 release. For later releases, we may return testsuite back but only when golang guys fix the licensing issue upstream.

description: updated
Revision history for this message
Martin Packman (gz) wrote :

I filed <https://github.com/golang/go/issues/10398> upstream for the unclear w3c licence issue.

Revision history for this message
Curtis Hovey (sinzui) wrote :

https://github.com/juju/juju/blob/master/scripts/win-installer/LICENCE.txt exists because the win-installer script create a window's installed and includes that licence in its distribution.

description: updated
Revision history for this message
Oleg Strikov (strikov-deactivatedaccount) wrote :

Bug description updated to track progress on (0) and (5)

description: updated
Curtis Hovey (sinzui)
tags: added: packaging tech-debt
Revision history for this message
Curtis Hovey (sinzui) wrote :

Hi Martin. I volunteered us to fix the text of the license issues.

Changed in juju-core:
assignee: nobody → Martin Packman (gz)
Revision history for this message
Martin Packman (gz) wrote :

Okay,

(1) charm
v4 <https://github.com/juju/charm/pull/118>
v5 <https://github.com/juju/charm/pull/119>

(2) cmd
<https://github.com/juju/cmd/pull/18>

(3) juju itself
changed, will push with the dep updates for the other branches

(4) testing
1.23 <https://github.com/juju/testing/pull/63>
master <https://github.com/juju/testing/pull/62>

(6) kisielk empty dir
Already drive-by fixed in the other juju-release-tools branch.

Revision history for this message
Martin Packman (gz) wrote :

Those dep updates and the header fixes in juju itself, for 1.23:

<https://github.com/juju/juju/pull/2061>

Changed in juju-core:
status: Triaged → In Progress
Revision history for this message
Martin Packman (gz) wrote :

All done. Upstream has a proposed fix for the unclear wc3 testcase licences too:

<https://go-review.googlesource.com/#/c/8978/>

Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.