Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nmap: 7.80 -> 7.91 #118158

Merged
merged 2 commits into from Jul 28, 2021
Merged

nmap: 7.80 -> 7.91 #118158

merged 2 commits into from Jul 28, 2021

Conversation

holzingk
Copy link
Contributor

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • [x ] macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@Ma27
Copy link
Member

Ma27 commented Mar 31, 2021

nmap-unfree is already at 7.91, nmap should stay at 4.80 for now due to licensing issues.

@holzingk
Copy link
Contributor Author

See also: #105119 and last paragraph of https://nmap.org/npsl/

TL;DR: They re-licensed version 7.90 and 7.91 under the old license. So I believe at least for this particular version the licensing issues are resolved.

@Ma27
Copy link
Member

Ma27 commented Mar 31, 2021

Fair enough, sorry. Reopening.

I guess in this case we can drop nmap-unfree though and make it an alias to nmap?

@Ma27 Ma27 reopened this Mar 31, 2021
@holzingk
Copy link
Contributor Author

I guess in this case we can drop nmap-unfree though and make it an alias to nmap?

As far as I understand the licensing remains unclear for future releases. Maybe just leave those two in parallel for now and let's see whether nmap-unfree will be required.

@Ma27
Copy link
Member

Ma27 commented Mar 31, 2021

I'm confused why we even have copied one expression there. Aliasing should be fine for now, if we license thing re-occurs, we can only make the new version available at nmap-unfree by removing the alias.

@r-rmcgibbo
Copy link

Result of nixpkgs-review pr 118158 at e12bbb6 run on aarch64-linux 1

8 packages built successfully:
  • crowbar
  • gnome3.gnome-nettool
  • nmap
  • nmap-graphical
  • python38Packages.python-nmap
  • python39Packages.python-nmap
  • rmount
  • rustscan

@fabaff
Copy link
Member

fabaff commented Mar 31, 2021

With the statement on the website, we no longer need nmap-unfree. I'm not sure that one sentence or paragraph without date or other reference, five months after the release, qualifies as re-licensing. The license file that is part of the 7.91 source says NPSL and the file headers also says NPSL. The source for 7.91 doesn't contain any details about the previous license or that's allowed to use it under GPLv2.

@holzingk
Copy link
Contributor Author

holzingk commented Apr 6, 2021

Also Debian sid included nmap 7.91 again.

It is true that the sources still mention the NPSL. GPLv2 is just an alternative license that we are allowed to use.

@fabaff
Copy link
Member

fabaff commented Apr 7, 2021

I assume that the Debian legal team has reviewed the current situation and if they agrees that the available statements are enough, it's fine for me.

@packetizeme
Copy link
Contributor

Worth noting that nmap 7.80 is currently broken on MacOS:

Starting Nmap 7.80 ( https://nmap.org ) at 2021-05-25 10:00 PDT
Assertion failed: (res > 7), function nsock_library_initialize, file nsock_pool.c, line 307.

This is fixed in 7.91 per nmap/nmap#2081

@SuperSandro2000 SuperSandro2000 changed the title Update nmap to 7.91 nmap: 7.80 -> 7.91 May 27, 2021
@SuperSandro2000
Copy link
Member

I assume that the Debian legal team has reviewed the current situation and if they agrees that the available statements are enough, it's fine for me.

With the version number.

7.91+dfsg1+really7.80+dfsg1-2

@fpletz fpletz merged commit e5209d3 into NixOS:master Jul 28, 2021
This was referenced Jul 28, 2021
@fabaff fabaff mentioned this pull request Aug 31, 2021
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants