Page MenuHomeSolus

Initial inclusion of nanorc
ClosedPublic

Authored by abdulocracy on Oct 14 2021, 11:26 AM.
Tags
None
Referenced Files
F10828978: D12019.id29193.diff
Sun, Jun 4, 10:51 AM
F10828441: D12019.id30827.diff
Sun, Jun 4, 9:25 AM
F10827931: D12019.id29244.diff
Sun, Jun 4, 7:57 AM
F10827649: D12019.id29229.diff
Sun, Jun 4, 7:11 AM
F10784432: D12019.id.diff
Fri, May 26, 12:52 AM
F10752642: D12019.id29220.diff
Tue, May 16, 3:41 PM
F10752621: D12019.diff
Tue, May 16, 3:37 PM
F10752608: D12019.id30827.diff
Tue, May 16, 3:35 PM

Details

Summary

Contains the scopatz nanorc files, new dependency of nano.

Test Plan

Edit .conf file with nano and see if trailing spaces are highlighted green.

Diff Detail

Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 2017
Build 2017: arc lint + arc unit

Event Timeline

JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
11

Let's just do: Package with language-specific nanorc files [...]

13

Use install instead

This revision now requires changes to proceed.Oct 15 2021, 9:40 AM
JoshStrobl added inline comments.
package.yml
13

Should be 00644

This revision now requires changes to proceed.Oct 15 2021, 11:27 AM
package.yml
13

It's a directory, so 755 I think.

DataDrake added a subscriber: DataDrake.

The name of this package should be nanorc to match the upstream repo.

This revision now requires changes to proceed.Oct 16 2021, 1:17 AM

Should it? Perhaps something like scopatz-nanorc since nanorc by itself is generic. I understand sticking to the upstream name is important but in this case the upstream name isn't really a name.

I don't see how it isn't a name? It's a repo full of nanorc files. Arch is the only distro that currently ships these using that name and I don't agree with their decision.

@DataDrake does raise a good point that nano-syntax-highlighting is not the correct upstream name and we shouldn't be basing our package names around what other operating systems do. nanorc makes sense to me as well.

abdulocracy retitled this revision from Initial inclusion of nano-syntax-highlighting to Initial inclusion of nanorc.Oct 16 2021, 8:35 PM
This revision is now accepted and ready to land.Jan 14 2022, 7:48 PM
This revision was automatically updated to reflect the committed changes.