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
Apache Flume Initial Package 1.9.0 #58728
Conversation
@@ -8580,6 +8580,9 @@ in | |||
apacheKafka_2_1 = callPackage ../servers/apache-kafka { majorVersion = "2.1"; }; | |||
apacheKafka_2_2 = callPackage ../servers/apache-kafka { majorVersion = "2.2"; }; | |||
|
|||
apacheFlume = apacheFlume_1_9_0; | |||
apacheFlume_1_9_0 = callPackage ../servers/apache-flume { majorVersion = "1.9.0"; }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think there will be reasons to keep multiple versions of flume?
It does not seem like there are multiple maintained at the same time.
If that's not the case, it'd be better not to have to maintain multiple versions for us.
homepage = http://flume.apache.org; | ||
description = "Flume is a distributed, reliable, and available service for efficiently collecting, aggregating, and moving large amounts of log data."; | ||
license = licenses.asl20; | ||
maintainers = [ ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you be willing to look after this package, i.e. be maintainer?
''; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = http://flume.apache.org; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
homepage = http://flume.apache.org; | |
homepage = "https://flume.apache.org"; |
We'll deprecate unquoted URL syntax, once RFC 45 is approved. Also their site supports https.
wrapProgram $out/bin/flume-ng \ | ||
--set FLUME_HOME "$out" \ | ||
--set JAVA_HOME "${jre}" | ||
chmod +x $out/bin\/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure wrapProgram
already marks its output as executable. Please try to remove it.
|
||
stdenv.mkDerivation rec { | ||
version = "${flumeVersion}"; | ||
name = "apache-flume-${version}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name = "apache-flume-${version}"; | |
pname = "apache-flume"; |
pname
appends the version name on its own.
with versionMap.${majorVersion}; | ||
|
||
stdenv.mkDerivation rec { | ||
version = "${flumeVersion}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version = "${flumeVersion}"; | |
version = flumeVersion; |
No string quoting necessary here :)
|
||
meta = with stdenv.lib; { | ||
homepage = http://flume.apache.org; | ||
description = "Flume is a distributed, reliable, and available service for efficiently collecting, aggregating, and moving large amounts of log data."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
8350d1f
to
54e22ec
Compare
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
@polynomial why didn't you respond to the suggestions? |
Closing due to being stale and no response from the author in a longer time. |
Motivation for this change
We use Apache Flume in production and are migrating everything to NixOS. I will write a module next, but this gets me a working Apache Flume jar, which is a great first step!
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)