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

tomcat5: remove (EOL) #18341

Merged
merged 1 commit into from
Sep 6, 2016
Merged

tomcat5: remove (EOL) #18341

merged 1 commit into from
Sep 6, 2016

Conversation

groxxda
Copy link
Contributor

@groxxda groxxda commented Sep 6, 2016

Motivation for this change

Please also backport to 16.09
"Fixes" #18336
The source for 5.0.27 is not available anymore. We could update to 5.0.30 (2004-11-24) or 5.5.36 (2012-10-09) but I don't think it's worth the effort.

Archive url: http://archive.apache.org/dist/tomcat/tomcat-5/
EOL announcement: https://tomcat.apache.org/tomcat-55-eol.html

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • OS X
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

Source is not available anymore.
@mention-bot
Copy link

@groxxda, thanks for your PR! By analyzing the annotation information on this pull request, we identified @eelcovisser and @edolstra to be potential reviewers

@chris-martin
Copy link
Contributor

Alternatively, #18345 actually fixes it.

@groxxda
Copy link
Contributor Author

groxxda commented Sep 6, 2016

@chris-martin : https://tomcat.apache.org/security-5.html

If there really is a need for tomcat 5, we should update to 5.5.36 and take the 10 years worth of security updates. Please note: even that version hasn't been fixed in nearly 4 years.

@chris-martin
Copy link
Contributor

The tomcat5 package doesn't even build, so I'm going to go ahead and agree with you, let's just ditch it.

@fpletz fpletz merged commit 36e8df1 into NixOS:master Sep 6, 2016
fpletz pushed a commit that referenced this pull request Sep 6, 2016
Source is not available anymore.
(cherry picked from commit 36e8df1)
fpletz pushed a commit that referenced this pull request Sep 6, 2016
Source is not available anymore.
(cherry picked from commit 36e8df1)
@groxxda groxxda deleted the tomcat5 branch September 8, 2016 16:45
adrianpk added a commit to adrianpk/nixpkgs that referenced this pull request May 31, 2024
Source is not available anymore.
(cherry picked from commit 36e8df1)
adrianpk added a commit to adrianpk/nixpkgs that referenced this pull request May 31, 2024
Source is not available anymore.
(cherry picked from commit 36e8df1)
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.

4 participants