FS#8004 - Nano pkgbuild syntax highlighting

Attached to Project: Arch Linux
Opened by . (Thralas) - Wednesday, 12 September 2007, 17:55 GMT
Last edited by Andreas Radke (AndyRTR) - Sunday, 30 December 2007, 09:54 GMT
Task Type Feature Request
Category Packages: Current
Status Closed
Assigned To Andreas Radke (AndyRTR)
Architecture All
Severity Very Low
Priority Normal
Reported Version 2007.08 Don't Panic
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

The syntax highlighting of nano for pkgbuilds is missing a few makepkg options.

I've attached a patch for /etc/nanorc which adds coloring for arch, options, noextract and the SHA sum options (like md5sums).
   nanorc-highlighting.patch (1.6 KiB)
This task depends upon

Closed by  Andreas Radke (AndyRTR)
Sunday, 30 December 2007, 09:54 GMT
Reason for closing:  Fixed
Additional comments about closing:  added nanorc to the backup array. reverted back to the default upstream nanorc.sample.
an updated and fixed pkg is in testing for signoff.
Comment by paul (pwt) - Saturday, 15 September 2007, 22:00 GMT
Please don't go further. The nano customization makes it unreadable using terminal emulations with a white background.

Considering this kind of add-ons violate the KISS mantra, I am asking you remove the nanorc customization from the package or include it as a separate file and print the way to install it (optionnaly) during install.

Of course, if you go on, I will continue disabling it with a NoUpgrade option in pacman.conf, but I should not be obliged to do so.

Paul
Comment by Roman Kyrylych (Romashka) - Sunday, 16 September 2007, 09:35 GMT
@ Andy: please add etc/nanorc to backup array.
Comment by Anders Bergh (anders) - Friday, 12 October 2007, 19:00 GMT
Please don't modify nanorc in this way, it's way too different from the example nanorc and I find it quite annoying. I think the package should simply install the nanorc.example as /etc/nanorc and then add it to the backup array.

Loading...