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

Migrate away from the "rip" command #21

Closed
baldurmen opened this issue May 19, 2016 · 6 comments
Closed

Migrate away from the "rip" command #21

baldurmen opened this issue May 19, 2016 · 6 comments
Labels
Accepted Accepted issue on our roadmap Design Design or UX/UI related
Milestone

Comments

@baldurmen
Copy link
Contributor

Hi!

I've been thinking for a while now that morituri's rip command is pretty badly named. Normally, when you download a program you kinda expect to be able to call it by it's name and not some other name!

Since you decided to fork morituri, I find it even more confusing, since both of them now use the same utility name...

Why don't you migrate away from it and use whipper as the way to call the program?

@JoeLametta
Copy link
Collaborator

JoeLametta commented Jun 12, 2016

Sure, this needs to be done... (it's a long time plan)
The migration would also take care of improving whipper's command parsing structure. Regarding this topic I received the following suggestion I'd like to use as reference:

rip cd rip -> whipper rip /dev/srX
rip cd info -> whipper info /dev/srX
whipper analyze /dev/srX (it would combine rip offset find and rip drive analyze)
The /dev/srX argument would be optional.

@baldurmen
Copy link
Contributor Author

baldurmen commented Jun 13, 2016

I think specifying the drive is a wonderful idea, especially for people
who (like I) use a multi-drive setup to save time.

On 2016-06-12 13:10, JoeLametta wrote:

Sure, this needs to be done... (it's a long time plan)
The migration would also take care of improving whipper's command parsing structure. Regarding this topic I received the following suggestion I'd like to use as reference:

rip cd rip -> whipper rip /dev/srX
rip cd info -> whipper info /dev/srX
whipper analyze /dev/srX (it would combine rip offset find and rip drive analyze)
The /dev/srX argument would be optional.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#21 (comment)

@JoeLametta JoeLametta added this to the 2.0 milestone Aug 21, 2016
@Voidi
Copy link

Voidi commented Sep 19, 2016

I totally agree with renaming the rip command to the project names, but why renaming the project?
Assuming the original the original author has dropped the project, this version would be the only one and nobody would be confused. And i really like the joke with name morituri.

@JoeLametta
Copy link
Collaborator

@Voidi Well that should be discussed with @thomasvs but, at the moment, I think it's better this way (less confusing). I've not really planned it thoroughly for the long term (definitive name is still undecided) but, even in the perspective of package maintainers^, using a different name could be a good thing.

Thanks for raising this point, hopefully others will comment on this one too.

^ When milestone 1.0 will be reached I'll try to get it officially packaged (starting with Debian, Ubuntu, Fedora).

@xmixahlx
Copy link

Whipper seems like a true fork. Unless Morituri is actually defunct, it
should be renamed. I vote Whipper.

It will be way easier to package, also.

On Mon, Sep 19, 2016 at 2:56 AM, JoeLametta [email protected]
wrote:

@Voidi https://github.com/Voidi Well that should be discussed with
@thomasvs https://github.com/thomasvs but, at the moment, I think it's
better this way (less confusing). I've not really planned it thoroughly for
the long term (definitive name is still undecided) but, even in the
perspective of package maintainers^, using a different name could be a good
thing.

Thanks for raising this point, hopefully others will comment on this one
too.

^ When milestone 1.0 will be reached I'll try to get it officially
packaged (starting with Debian, Ubuntu, Fedora).


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#21 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATsw5ks_tVpw2-jAiJbdjlhvZRTOWqYQks5qrlxYgaJpZM4Ih5xe
.

@RecursiveForest
Copy link
Contributor

I +1 calling the executable whipper since we can't seem to find a better project name.

@JoeLametta JoeLametta added Accepted Accepted issue on our roadmap Design Design or UX/UI related and removed enhancement labels Nov 12, 2018
@JoeLametta JoeLametta modified the milestones: 2.0, 1.0 Nov 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Design Design or UX/UI related
Projects
None yet
Development

No branches or pull requests

5 participants