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

activator: remove package from package-set #25616

Merged
merged 1 commit into from
May 10, 2017
Merged

activator: remove package from package-set #25616

merged 1 commit into from
May 10, 2017

Conversation

Ma27
Copy link
Member

@Ma27 Ma27 commented May 8, 2017

Motivation for this change

⚠️ Caution: BC break: this change removes a package which might break the setup of several people!

the typesafe activator is about to reach end of life at the end of this
month and the servers will be decommissioned at the end of 2017.

See http://www.lightbend.com/community/core-tools/activator-and-sbt for
and the project's readme
(https://github.com/typesafehub/activator/blob/master/README.md) for further reference.

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
    • macOS
    • 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.

@mention-bot
Copy link

@Ma27, thanks for your PR! By analyzing the history of the files in this pull request, we identified @edwtjo, @cko and @NeQuissimus to be potential reviewers.

@Ma27
Copy link
Member Author

Ma27 commented May 8, 2017

btw it seems as the XCode build fails because of a failed download, but that should be unrelated to my change, shouldn't it?
(see https://travis-ci.org/NixOS/nixpkgs/jobs/230018721)

@@ -6320,8 +6320,6 @@ with pkgs;

### DEVELOPMENT / TOOLS

activator = callPackage ../development/tools/activator { };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if we have a general policy on this, but I generally appreciate it when instead of removing derivation attributes, we replace them with something like throw "Typesafe Activator was removed in X for reason Y. See https://github.com/NixOS/nixpkgs/pull/25616 for more information", and then eventually we delete the attribute. This gives people a way to figure out what went wrong (although nix-env will just break silently, but fixing that is more work) even if they can't get the package.

I think there are probably better ways to do this in a more systematic way, but until we've agreed on those, I'd prefer a throw

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done. It gives me now the following output when trying to build:

→ nix-build -A activator
error: Typesafe Activator was removed in 2017-05-08 as the actual package reaches end of life.

See https://github.com/NixOS/nixpkgs/pull/25616
and http://www.lightbend.com/community/core-tools/activator-and-sbt
for more information.

I've squashed it into the previous commit as it directly belongs to this change.

the typesafe activator is about to reach end of life at the end of this
month (2017-05) and the servers will be decommissioned at the ond of 2017.

See http://www.lightbend.com/community/core-tools/activator-and-sbt for
and the project's readme
(https://github.com/typesafehub/activator/blob/master/README.md) for further reference.
@Ma27
Copy link
Member Author

Ma27 commented May 8, 2017

well...why does travis try to build the openjdk? This will take years...

@Ma27
Copy link
Member Author

Ma27 commented May 9, 2017

@copumpkin anything else tbd?

@fpletz
Copy link
Member

fpletz commented May 10, 2017

👍

@fpletz fpletz merged commit f2e100c into NixOS:master May 10, 2017
@Ma27 Ma27 deleted the package/activator/remove branch May 10, 2017 04:11
@peti peti mentioned this pull request May 15, 2017
7 tasks
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.

None yet

4 participants