[Bug 990932] Review Request: erlang-riaknostic - A diagnostic tool for Riak installations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=990932



--- Comment #5 from Peter Lemenkov <lemenkov@xxxxxxxxx> ---
Sorry, submitted too fast.

(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)

> erlang-riaknostic.src: W: invalid-url Source0:
> basho-riaknostic-v1.1.0-0-g8cfb08e.tar.gz → You can use github link like
> https://github.com/basho/riaknostic/archive/v1.1.0.tar.gz, which should give
> the same file.

Fixed.

> Also, there's a new upstream release v1.2.0.

Yes, I'm aware of this. But I can't use 1.2.0 until I upgrade to Riak 1.4.x (I
plan to update in Fedora 21).

> Is the package really usable without riak-admin? The documentation mentions
> that it can be run standalone for testing purposes... I don't know almost
> anything about riak, so I'm guessing here, but I think that it should have
> Requires:riak.

It's not a standalone package - it's a library. README doesn't properly
describes that.

riak-admin is the script from riak package which could use riaknostic if this
library is available. So it's  riak who is dependent upon riaknostic, not the
opposite.

> Also, maybe the description could be updated to say "is a plugin for
> riak-admin" instead of "is integrated with ...", just to be more direct.

Yes, definitely!

New package:

* http://peter.fedorapeople.org/erlang-riaknostic.spec
* http://peter.fedorapeople.org/erlang-riaknostic-1.1.0-1.fc20.src.rpm


(In reply to Christopher Meng from comment #2)
> Should we add VCS tag? I think rpm doesn't support checkout from git and
> install, not sure if we need that.

I added this mostly for the reference.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Aa4pODGNa3&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]