Add -e option: Interrupt via Ctrl+C, Proper Makefile, and Cleanup sl.h

mtoyoda / sl

SL(1): Cure your bad habit of mistyping

Other

2.95k stars 427 forks source link

Add -e option: Interrupt via Ctrl+C, Proper Makefile, and Cleanup sl.h #17

Closed ryanmjacobs closed 7 years ago

ryanmjacobs commented 10 years ago

Add -e option to allow interrupt via Ctrl+C. So users can, alias sl='sl -e' if they don't want to wait each time for sl to finish running.

Makefile now includes: install, uninstall, and clean. Therefore, users can install sl by running:

make
make install

Add header guards to sl.h, and move function prototypes to sl.h.
README.ja.md and sl.1.ja still need to be updated (I don't know Japanese).

mtoyoda commented 10 years ago

Sorry for replying late. The -e option is a local extension for the debian package. I'm still wondering that users really need the option.

ryanmjacobs commented 10 years ago

That's fine if you don't merge it. I just created this fork because I wanted the -e option on my system. If other users want it too, they can grab it from my fork.

jeandrek commented 8 years ago

I get impatient because I mistype ls as sl very often :stuck_out_tongue:

jeandrek commented 8 years ago

Hm. I applied this patch and the manpage has changed and the source does say it's changed but -e doesn't work...

ryanmjacobs commented 8 years ago

@Jonathan50 Sorry for the late reply. My inbox has been stuffed lately.

Can you please try these commands in order?

$  git clone https://github.com/ryanmjacobs/sl
$ cd sl
$ make
$ ./sl -e

jeandrek commented 8 years ago

It works. But if I modify the PKGBUILD to apply the patch it doesn't work.

ryanmjacobs commented 8 years ago

Can you try this PKGBUILD?

(I basically took sl-git, and switched out mtoyoda with ryanmjacobs)

jeandrek commented 8 years ago

It works great thanks