FS#71829 - [maven] profile.d file should not be installed as executable

Attached to Project: Community Packages
Opened by Emanuele Torre (emanuele6) - Saturday, 14 August 2021, 01:35 GMT
Last edited by Buggy McBugFace (bugbot) - Saturday, 25 November 2023, 20:02 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Levente Polyak (anthraxx)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

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
This task depends upon

Closed by  Buggy McBugFace (bugbot)
Saturday, 25 November 2023, 20:02 GMT
Reason for closing:  Moved
Additional comments about closing:  https://gitlab.archlinux.org/archlinux/p ackaging/packages/maven/issues/1

Loading...