Fix name of mcedit #2524

Merged
merged 1 commit into from Dec 9, 2016

Conversation

Projects
None yet
4 participants
@jwilk
Contributor

jwilk commented Oct 31, 2016

It's mcedit, not mc-edit.

Fix name of mcedit
It's "mcedit", not "mc-edit".
@stapelberg

This comment has been minimized.

Show comment
Hide comment
@stapelberg

stapelberg Oct 31, 2016

Member

Could it be that some distributions ship it as mc-edit? Should we have both here?

Member

stapelberg commented Oct 31, 2016

Could it be that some distributions ship it as mc-edit? Should we have both here?

@jwilk

This comment has been minimized.

Show comment
Hide comment
@jwilk

jwilk Oct 31, 2016

Contributor

I doubt mc-edit is a thing. There's exactly one hit on DCS (i3 itself), no hits on RPM search; I couldn't find anything relevant on Google either.

@hboetes, it was apparently you who added mc-edit to the list of editors (ac3541c).
Was it a typo for mcedit or is there actually a distro that ships it under this name?

Contributor

jwilk commented Oct 31, 2016

I doubt mc-edit is a thing. There's exactly one hit on DCS (i3 itself), no hits on RPM search; I couldn't find anything relevant on Google either.

@hboetes, it was apparently you who added mc-edit to the list of editors (ac3541c).
Was it a typo for mcedit or is there actually a distro that ships it under this name?

@hboetes

This comment has been minimized.

Show comment
Hide comment
@hboetes

hboetes Oct 31, 2016

I can’t look it up right now, but the intention was the editor that is
provided by the midnight commander package.

hboetes commented Oct 31, 2016

I can’t look it up right now, but the intention was the editor that is
provided by the midnight commander package.

@Airblader

This comment has been minimized.

Show comment
Hide comment
@Airblader

Airblader Dec 2, 2016

Member

@jwilk @hboetes What's the status here?

Member

Airblader commented Dec 2, 2016

@jwilk @hboetes What's the status here?

@hboetes

This comment has been minimized.

Show comment
Hide comment
@hboetes

hboetes Dec 2, 2016

hboetes commented Dec 2, 2016

@Airblader

This comment has been minimized.

Show comment
Hide comment
@Airblader

Airblader Dec 2, 2016

Member

@stapelberg Then this should be good to go.

Member

Airblader commented Dec 2, 2016

@stapelberg Then this should be good to go.

@stapelberg stapelberg merged commit 8d739b7 into i3:next Dec 9, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment