Ticket #388 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

*.tcl installed as executable

Reported by: hamish Owned by: grass-dev@…
Priority: minor Milestone: 6.4.0
Component: Installation Version: svn-develbranch6
Keywords: build install Cc:
Platform: Linux CPU: Unspecified

Description

Hi,

browsing through the Debian lintian package report for grass 6.2.3 I notice that the .tcl files installed in $(ETC)/dm/,gm/ are incorrectly given an executable bit in the file system.

 http://lintian.debian.org/maintainer/pkg-grass-devel@lists.alioth.debian.org.html#grass

some Makefile magic required..

Hamish

Change History

in reply to: ↑ description ; follow-up: ↓ 2   Changed 3 years ago by glynn

Replying to hamish:

browsing through the Debian lintian package report for grass 6.2.3 I notice that the .tcl files installed in $(ETC)/dm/,gm/ are incorrectly given an executable bit in the file system.

some Makefile magic required..

I've changed gis.m/Makefile from $(INSTALL) to $(INSTALL_DATA) in r34721. tksys.tcl is an executable script, so I explicitly added execute permission for that file.

d.m would need something similar, if we're going to keep it. Is there any reason to? Does it still work?

in reply to: ↑ 1   Changed 3 years ago by hamish

Replying to glynn:

I've changed gis.m/Makefile from $(INSTALL) to $(INSTALL_DATA) in r34721. tksys.tcl is an executable script, so I explicitly added execute permission for that file.

thanks.

d.m would need something similar, if we're going to keep it. Is there any reason to?

Apparently some people prefer it. All the g.gui and init.sh stuff is set up to handle it well in a mixed environment; it doesn't hurt anything; and has taken minimal effort to maintain. So I'd say there's no reason to remove it.

Does it still work?

yes, it works fine.

Hamish

  Changed 3 years ago by hamish

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.