Skip to content

Relicense files in doc/tls/notebook/raw_data#4542

Merged
gpotter2 merged 1 commit intomasterfrom
gpotter2-patch-1
Sep 30, 2024
Merged

Relicense files in doc/tls/notebook/raw_data#4542
gpotter2 merged 1 commit intomasterfrom
gpotter2-patch-1

Conversation

@gpotter2
Copy link
Copy Markdown
Member

Spotted by debian.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.61%. Comparing base (41b6f2c) to head (fe2a02e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4542      +/-   ##
==========================================
- Coverage   81.61%   81.61%   -0.01%     
==========================================
  Files         356      356              
  Lines       85533    85533              
==========================================
- Hits        69812    69804       -8     
- Misses      15721    15729       +8     

see 6 files with indirect coverage changes

@gpotter2
Copy link
Copy Markdown
Member Author

@charles2910 FYI.

@gpotter2 gpotter2 closed this Sep 30, 2024
@gpotter2 gpotter2 reopened this Sep 30, 2024
@gpotter2 gpotter2 merged commit 1f9061d into master Sep 30, 2024
@gpotter2 gpotter2 deleted the gpotter2-patch-1 branch September 30, 2024 10:24
@charles2910
Copy link
Copy Markdown
Contributor

@gpotter2 thanks! I'm excluding the doc/ directory entirely (almost entirely, I'm keeping the manpage and logo), so this change doesn't affect Debian.

@gpotter2
Copy link
Copy Markdown
Member Author

gpotter2 commented Oct 20, 2024

Sorry, I wasn't very clear ^^

I was talking about https://salsa.debian.org/pkg-security-team/scapy/-/blob/debian/master/debian/patches/Remove-tests-dependent-on-non-free-data.patch?ref_type=heads

Tests should be able to run entirely under GPLv2, which is why I relicensed those files. You should now no longer need this patch :)

Thanks for your time

@charles2910
Copy link
Copy Markdown
Contributor

Thank you actually! I will include that folder and drop the patch in the next scapy release!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants