search

clustericious / Yars

Yet-another ARchive Service
5 stars 2 forks source link

Initial commit of yars_exercise #22

Closed CurtTilmes closed 8 years ago

CurtTilmes commented 8 years ago

Still needs some polish, but you might find it handy to play with...

plicease commented 8 years ago

This adds a few dependencies that I am not sure that I want to in the server / client package.

CurtTilmes commented 8 years ago

Any more details? Which dependencies don't you like and why?

Do you just think this program shouldn't be there at all? (It seems like anyone installing a Yars server should have something like this to exercise their install and verify it prior to use.)

If it is just the dependencies, I generally go for well-tested, independently supported, modular solutions, easily called, but most of those dependencies could easily be removed by embedding more complicated, more error-prone code directly into the program if you think that's a better approach. ;-)

plicease commented 8 years ago

I am not against dependencies however, there are a couple in this PR which were problematic:

  1. Bytes::Random is not installed on current prod machines, and the SAs have frozen the packages there.
  2. Number::Format replicates capability from Number::Bytes::Human already used by Yars. I can see arguments for both (N::B::H has a bug reported on rt that is quite disturbing, N::F doesn't support tera (T)).

I'd actually like to have something like this for QA to play around with, and I hope to rework it into something usable.