profile.d file should not be installed as executable

Task Info (Flyspray)
Opened By Emanuele Torre (emanuele6)
Task ID 71829
Type Bug Report
Project Community Packages
Category Packages
Version None
OS All
Opened 2021-08-14 01:35:34 UTC
Status Assigned
Assignee Levente Polyak (anthraxx)

Details

Description: I know community/maven is not the only package that installed /etc/profile.d files as exceutables, but, since it is not necessary (and it does not make much sense to run a script that only sets environments variable in its own enviroment), it would be nice if it did not.

This like of the PKGBUILD file should be changed from:

 install -Dm 755 ../maven.sh -t "${pkgdir}/etc/profile.d"

to:

 install -Dm 644 ../maven.sh -t "${pkgdir}/etc/profile.d"

Most other packages (core/perl, core/gawk, ...) use 644 which I also think is the best set of permission for /etc/profile.d files.

Additional info:

  • community/maven package version: 3.8.1-1

Steps to reproduce: $ stat /etc/profile.d/maven.sh