#86 Remove .sh extension from scripts.

[Giulio Paci]

Several scripts are distributed with sptk. They all have a .sh extension (despite eing csh scripts).
It is not a good practice to put extensions on this kind of scripts (users are not interested in the language the command are written) and it is confusing if the extension is not the right one.

Attached is a patch that removes the extensions from scripts and simplify the Makefile.am code handling scripts installation.

[Giulio Paci]

Hi, I just noticed that this issue has been closed and no action has been taken.
Should I consider this issue rejected? If yes, can you please comment why?

Bests,
Giulio.

[Akira Tamamori]

Hi,

From our discussion, we thought it is sufficient that symbolic links remain and *.sh themselves are put in somewhere (not bindir).

By the way, in your environment, 'csh' is installed?

[Giulio Paci]

As far as I remember I am using tcsh. And I have to say that the scripts
are working for me.
It is just that having language extensions in scripts is usually a bad
practice.
Il 27/nov/2013 01:04 "Akira Tamamori" mataki@users.sf.net ha scritto:

Hi,

From our discussion, we thought it is sufficient that symbolic links
remain and *.sh themselves are put in somewhere (not bindir).

By the way, in your environment, 'csh' is installed?

• [feature-requests:#86]
  http://sourceforge.net/p/sp-tk/feature-requests/86/ Remove .sh extension
  from scripts.*

Status: closed
Created: Fri Oct 04, 2013 03:04 AM UTC by Giulio Paci
Last Updated: Tue Nov 26, 2013 04:33 PM UTC
Owner: nobody

Several scripts are distributed with sptk. They all have a .sh extension
(despite eing csh scripts).
It is not a good practice to put extensions on this kind of scripts (users
are not interested in the language the command are written) and it is
confusing if the extension is not the right one.

Attached is a patch that removes the extensions from scripts and simplify
the Makefile.am code handling scripts installation.

---

Sent from sourceforge.net because you indicated interest in
https://sourceforge.net/p/sp-tk/feature-requests/86/

To unsubscribe from further messages, please visit
https://sourceforge.net/auth/subscriptions/

[Akira Tamamori]

Re-opened for discussion.