Copyright confusion about adblock-rust

[penguin359]

OK, all fixes have been applied. Package is ready for submission to the Debian project.

[penguin359]

Actually, I now realize that data/ is in the exclude list and I was running my license scanner on the upstream repo instead of the crate source as distributed. If that folder is not included then there's no need for me to worry about including it in the copyright file. Debian prefers upstream tarballs to repositories and, for Rust, that generally means what's downloaded from crates.io. I do want to make sure I have the correct ownership attributed elsewhere as appropriate. If "The Brave Authors" properly describes the owner of the copyright, then that is what I should use.

The one file I do still see with an alternate copyright and license is src/url_parser/parser.rs. I don't see any other mention in other source files for that.

[rillian]

Looks like src/url_parser/parser.rs was forked from rust-url sometime in early 2019, but the repository doesn't preserve the exact original version. The original license and authorship applies to a substantial portion of that file.

[fmarier]

Link to the Debian bug for this: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1063626

[penguin359]

OK, for 0.8.6 I'll keep that in the copyright, but I can certainly remove that once a new release is out. Just add a comment to this ticket and I'll update the Debian packaging.

Also, I found one final file in question, src/resources/resource_storage.rs#L23. I've added a mention of it to the Debian copyright file for the moment. I'm not sure how to handle it exactly as it seems to have been modified from the original before it was committed to this repo and it seems to just cover a snippet of code added to an already existing file that was originally clean. I've marked it as best as I can in the Debian DEP-8 syntax. You can check it out here:

https://salsa.debian.org/penguin359/debcargo-conf/-/blob/add-adblock/src/adblock/debian/copyright

Once this is approved, we should be able to get adblock in as an official Debian package.

[penguin359]

Actually, I think I found one more file, src/url_parser/parser.rs which is:

Copyright (c) 2013-2016 The rust-url developers.

And licensed under ASL-2.0 or MIT.

[antonok-edm]

Hi @penguin359, thanks for working on that!

Everything you found so far looks accurate, although I'll add a couple of notes:

• Brave Software, Inc. licenses under Copyright (c) <year> The Brave Authors by default. Somehow this repo was originally created without that, although I assume it might make things easier for your task if it's possible to use it instead. adblock-rust was developed by both Andrius and I as part of our respective employment contracts with Brave; I just don't know if it's safe to retroactively edit that. I've asked internally.
• Now that I think about it, I suppose the files under data/ ought to have respective in-tree licenses. That being said, those files aren't intended to be distributed for the purposes of packaging (see Cargo.toml, .npmignore). It's just for regression testing during development. You're welcome to exclude those (if there is a way to do so).
• If you must include licenses for files under data/, be advised that there are a few other sources, including Easylist (licensing info), NoCoin (license), brave/adblock-lists (license), ABP Japanese filters (licenses?), Disconnect (GPLv3 prior to relicensing), Malware Domain List (no longer active... and I couldn't find a license on the archived version 😬), Spam404 (license), and the uBlock Origin list authors (license).

[antonok-edm]

@penguin359 I confirmed there should be no issues using The Brave Authors (along with the The rust-url developers).

I've also opened #333 to track removing the forked rust-url parts, which we probably don't want to keep around for too much longer, but please include that in the meantime.

Anything else needed?

[antonok-edm]

Also, I found one final file in question

Good catch, thanks! That looks correct to me.

Just a couple notes on the proposed copyright file you linked:

Upstream-Contact:
 Andrius Aucinas <[email protected]>
 Anton Lazarev <[email protected]>

Andrius no longer works at Brave; I don't think that inbox will get any responses. Probably best to just use my own email there. I originally left his info in the authorship metadata since I didn't want to take all the credit, although I suppose it makes sense to remove at least the email from there too.

Files: *
 Copyright: 2019-2020 The Brave Authors

probably 2019-2024?