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

Rename the snap to minio-client #2243

Merged
merged 1 commit into from
Aug 27, 2017
Merged

Rename the snap to minio-client #2243

merged 1 commit into from
Aug 27, 2017

Conversation

come-maiz
Copy link

In linux, mc is midnight commander.
In the traditional linux distro world, name conflicts are forbidden. Because we are trying to make this decentralized and self-serving, we need less strict rules. For this, we added the concept of aliases. Then the user can decide what application owns the mc name.

You could make a claim for the mc name, and keep it. But I think it would be less confusing for existing linux users to rename the snap and use mc as an alias.

I would love your feedback about this feature. Please let me know what do you think. Also, this needs somebody from your org to register the new name in build.snapcraft.io to get the continuous delivery working.

In linux, mc is midnight commander.
In the traditional linux distro world, name conflicts are forbidden. Because we are trying to make this decentralized and self-serving, we need less strict rules. For this, we added the concept of aliases. Then the user can decide what application owns the mc name.

You could make a claim for the mc name, and keep it. But I think it would be less confusing for existing linux users to rename the snap and use mc as an alias.

I would love your feedback about this feature. Please let me know what do you think. Also, this needs somebody from your org to register the new name in build.snapcraft.io to get the continuous delivery working.
@codecov-io
Copy link

Codecov Report

Merging #2243 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #2243   +/-   ##
======================================
  Coverage    9.36%   9.36%           
======================================
  Files          98      98           
  Lines        9193    9193           
======================================
  Hits          861     861           
  Misses       8199    8199           
  Partials      133     133

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd3b659...bdc00fe. Read the comment docs.

@harshavardhana
Copy link
Member

I would love your feedback about this feature. Please let me know what do you think. Also, this needs somebody from your org to register the new name in build.snapcraft.io to get the continuous delivery working.

Yes this change is fine since alias approach sounds easier.

Copy link
Member

@vadmeste vadmeste left a comment

Choose a reason for hiding this comment

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

LGTM

@deekoder deekoder merged commit 4189e40 into minio:master Aug 27, 2017
@come-maiz come-maiz deleted the patch-1 branch August 28, 2017 14:06
@come-maiz
Copy link
Author

Thanks for the merge :)

Now you need to register the new name. You can do it from https://dashboard.snapcraft.io, or with:

snapcraft register minio-client

harshavardhana pushed a commit that referenced this pull request Aug 31, 2017
In linux, mc is midnight commander. In the traditional
linux distro world, name conflicts are forbidden. Because
we are trying to make this decentralized and self-serving,
we need less strict rules. For this, snap has a concept
of aliases. Then the user can decide what application owns
the mc name.
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.

5 participants