After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 706667 - [patch] Fix permission denied error when installing from an nfs directory
[patch] Fix permission denied error when installing from an nfs directory
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: build
2.36.x
Other Mac OS
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2013-08-23 14:00 UTC by martin
Modified: 2017-11-03 20:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to create py files at build time (1007 bytes, patch)
2013-08-23 14:02 UTC, martin
committed Details | Review

Description martin 2013-08-23 14:00:58 UTC
If the build directory is mounted via nfs, then sudo make install will fail with the following error:

make  install-data-hook
  GEN      libglib-gdb.py
/bin/sh: ./libglib-gdb.py: Permission denied
make[5]: *** [libglib-gdb.py] Error 1
make[4]: *** [install-data-am] Error 2
make[3]: *** [install-am] Error 2
make[2]: *** [install-recursive] Error 1
make[1]: *** [install] Error 2
make: *** [install-recursive] Error 1

The problem is that install-data-hook tries to create the file libglib-gdb.py in the build directory at install-time, but this is not allowed when the uid is root.

A similar problem occurs with libgobject-gdb.py.

The attached patch fixes this by adding a makefile dependency to create the files at build-time.  The patch is against the release tar, hence it contains changes to Makefile.in rather than Makefile.am.
Comment 1 martin 2013-08-23 14:02:34 UTC
Created attachment 252862 [details] [review]
Patch to create py files at build time
Comment 2 Philip Withnall 2017-11-03 20:15:37 UTC
Review of attachment 252862 [details] [review]:

Looks reasonable.
Comment 3 Philip Withnall 2017-11-03 20:16:25 UTC
Turned it into a proper patch with some amount of attribution, and pushed.