Opened 3 years ago

Closed 2 years ago

#20759 closed task (fixed)

check if LICENSE file is up-to-date

Reported by: iwakeh Owned by: hiro
Priority: Low Milestone:
Component: Metrics Version:
Severity: Normal Keywords:
Cc: iwakeh Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

This was already done for Onionoo, open are

  • metrics-lib
  • CollecTor
  • ExoneraTor
  • Metrics web

Child Tickets

Attachments (4)

metrics_lib_license.patch (119.2 KB) - added by hiro 3 years ago.
LICENSE for included binaries in metrics-lib
collector_license.patch (126.7 KB) - added by hiro 3 years ago.
LICENSE for included binaries in collector
exonerator_license.patch (110.8 KB) - added by hiro 3 years ago.
LICENSE for included binaries in exonerator
update_license.patch (35.5 KB) - added by hiro 2 years ago.
Metrics-web patch

Download all attachments as: .zip

Change History (13)

comment:1 Changed 3 years ago by hiro

Hi Iwakeh, if you want I can take this up, since I have already done it for Onionoo.

comment:2 in reply to:  1 Changed 3 years ago by karsten

Cc: iwakeh added

Replying to hiro:

Hi Iwakeh, if you want I can take this up, since I have already done it for Onionoo.

Sounds like a fine plan to me. iwakeh, any objections?

comment:3 in reply to:  1 Changed 3 years ago by iwakeh

Replying to hiro:

Hi Iwakeh, if you want I can take this up, since I have already done it for Onionoo.

That would be great!

comment:4 Changed 3 years ago by hiro

Owner: changed from metrics-team to hiro
Status: newassigned

Changed 3 years ago by hiro

Attachment: metrics_lib_license.patch added

LICENSE for included binaries in metrics-lib

Changed 3 years ago by hiro

Attachment: collector_license.patch added

LICENSE for included binaries in collector

Changed 3 years ago by hiro

Attachment: exonerator_license.patch added

LICENSE for included binaries in exonerator

comment:5 Changed 3 years ago by iwakeh

Status: assignedneeds_review

@hiro: Thanks!

Setting ticket to 'needs_review', so the review is on the todo agenda.

comment:6 Changed 2 years ago by karsten

Status: needs_reviewnew

All three patches applied, thanks a lot! Looks like the only remaining license file is metrics-web's. Want to submit a patch for that, too?

comment:7 Changed 2 years ago by hiro

Hey Karsten,
Sure, was waiting for the website to be live to check the license again in case something had changed. Will get back to you with the license for metrics-web.

comment:8 Changed 2 years ago by hiro

Submitting patch for metrics web.

Changed 2 years ago by hiro

Attachment: update_license.patch added

Metrics-web patch

comment:9 Changed 2 years ago by karsten

Resolution: fixed
Status: newclosed

Loos good, applied! I guess that resolves this ticket. Closing. Thank you!

Note: See TracTickets for help on using tickets.