[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-07-31 Thread Sebastien Bacher
that one was promoted in bionic ** Changed in: ubuntu-report (Ubuntu) Status: New => Fix Released -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to ubuntu-report in Ubuntu. https://bugs.launchpad.net/bugs/1759540 Title: [MIR]

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-18 Thread Didier Roche
"Note thus that "golang-github-ubuntu-ubuntu-report-dev" should still be kept in main." -> "Note thus that "golang-github-ubuntu-ubuntu-report- dev" should still be kept in **universe**." of course. -- You received this bug notification because you are a member of Desktop Packages, which is

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-18 Thread Didier Roche
As discussed, ubuntu-report is now using its own upstream vendored dependencies corresponding to the ancient build-deps here. This is what upstream is already testing against. debian/copyright is updated (grepping manually for copyrights, as per Matthias' remark) and build system updated in

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-18 Thread Didier Roche
Reverting to "NEW" for second review (1.0.9) from the MIR team. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to ubuntu-report in Ubuntu. https://bugs.launchpad.net/bugs/1759540 Title: [MIR] ubuntu-report: send telemetry data to ubuntu

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-17 Thread Sebastien Bacher
The issue got discussed on IRC, as written in the description the upstream projects include a vendors/ subdir with those components, we didn't use that because we though using the distribution versions was the preferred way. We are going to switch back to use the vendors/ copies instead (which

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-17 Thread Steve Langasek
> We don't need the Go API "golang-github-ubuntu-ubuntu-report-dev" > which has many go package dependencies in universe. Unfortunately, listing golang-github-ubuntu-ubuntu-report-dev is not sufficient to keep the go tree out of main. ubuntu-report is itself implemented in go, which means it

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-17 Thread Steve Langasek
Override component to main ubuntu-report 1.0.7 in bionic: universe/utils -> main 1 publication overridden. Override component to main ubuntu-report 1.0.7 in bionic amd64: universe/utils/optional/100% -> main ubuntu-report 1.0.7 in bionic arm64: universe/utils/optional/100% -> main ubuntu-report

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-17 Thread Sebastien Bacher
The reviews state that the package is fine, the few tweaks that have been requested have been done, setting as fix commited ** Changed in: ubuntu-report (Ubuntu) Status: New => Fix Committed -- You received this bug notification because you are a member of Desktop Packages, which is

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-16 Thread Michael Hudson-Doyle
The package looks fine to me now. -- You received this bug notification because you are a member of Desktop Packages, which is subscribed to ubuntu-report in Ubuntu. https://bugs.launchpad.net/bugs/1759540 Title: [MIR] ubuntu-report: send telemetry data to ubuntu server Status in

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-10 Thread Didier Roche
Ok, uploaded 1.0.6 which address the above issue, stripping the vendor directory from the source tarball (and change debian/copyright), typo fixes, while shipping a symbols file which ignores private cgo symbols. Please let me know if any other changes are required, thanks! ** Changed in:

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-10 Thread Didier Roche
@doko: - the symbols file would be irrelevant. We tried it in previous uploads (see package), but any even non API breaking symbols generates different hash for internal symbols which starts with _ (and per architecture…). Would you have a way to tell dh_symbols to ignore all symbols starting

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-09 Thread Michael Hudson-Doyle
Doko asked me to look at the package and it seems OK to me on a quick glance. (Only spent a few minutes, let me know if you'd like a deeper review!) On the packaging side, I'm a bit surprised at the use of go:generate to create the shared lib but it works I guess. Another thing that's more on the

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-04-09 Thread Matthias Klose
- libsysmetrics1 is missing a symbols file - Package description s/contain/contains/ - the copyrights are incomplete. grep -ri copyright shows a lot more names. that includes the vendor files, but these are shipped in the source package. ** Changed in: ubuntu-report (Ubuntu) Status:

[Desktop-packages] [Bug 1759540] Re: [MIR] ubuntu-report: send telemetry data to ubuntu server

2018-03-29 Thread Didier Roche
** Description changed: Note that only the C library ("libsysmetrics1") and the command line ("ubuntu-report") packages are needed in main (we should also promote "libsysmetrics-dev" package, even if not necessary). We don't need the Go API "golang-github-ubuntu-ubuntu-report-dev" which