The install path should be `$DESTDIR/$PREFIX/<path>`. PREFIX should be the path under which the file is going to be installed. DESTDIR is a way to make it easier to package builders to collect installed files. It shouldn't be used as an alternative to prefix. And I think software generally shouldn't be run from DESTDIR. More information is available at https://www.gnu.org/software/automake/manual/html_node/DESTDIR.html On Thu, May 12, 2016 at 3:55 PM, Aravinda <avishwan@xxxxxxxxxx> wrote: > > regards > Aravinda > > On 05/12/2016 02:33 PM, Niels de Vos wrote: > > On Thu, May 12, 2016 at 02:01:43PM +0530, Aravinda wrote: > > I checked the Makefile.am and configure.ac of glupy, looks good to me. I > don't think we have issue in glupy. > > If we run make install with DESTDIR empty then > `${DESTDIR}/usr/lib/python2.6/site-packages/gluster` will become > /usr/lib/python2.6/site-packages/gluster. So we will get that error. > > For example, > DESTDIR= make install > or > make install DESTDIR= > > Can we check how we are executing smoke test? > > I think it is this script, no DESTDIR in there: > > > https://github.com/gluster/glusterfs-patch-acceptance-tests/blob/master/build.sh > > My guess is that the --prefix ./configure option is not honoured? > > DESTDIR will not get substitute during ./configure, it is used during make > install. Once we run ./autogen.sh and ./configure(with whatever prefix), > generated Makefile for glupy is > > > install-pyglupyPYTHON: $(pyglupy_PYTHON) > @$(NORMAL_INSTALL) > @list='$(pyglupy_PYTHON)'; dlist=; list2=; test -n "$(pyglupydir)" || > list=; \ > if test -n "$$list"; then \ > echo " $(MKDIR_P) '$(DESTDIR)$(pyglupydir)'"; \ > $(MKDIR_P) "$(DESTDIR)$(pyglupydir)" || exit 1; \ > fi; \ > for p in $$list; do \ > if test -f "$$p"; then b=; else b="$(srcdir)/"; fi; \ > if test -f $$b$$p; then \ > $(am__strip_dir) \ > dlist="$$dlist $$f"; \ > list2="$$list2 $$b$$p"; \ > else :; fi; \ > done; \ > for file in $$list2; do echo $$file; done | $(am__base_list) | \ > while read files; do \ > echo " $(INSTALL_DATA) $$files '$(DESTDIR)$(pyglupydir)'"; \ > $(INSTALL_DATA) $$files "$(DESTDIR)$(pyglupydir)" || exit $$?; \ > done || exit $$?; \ > if test -n "$$dlist"; then \ > $(am__py_compile) --destdir "$(DESTDIR)" \ > --basedir "$(pyglupydir)" $$dlist; \ > else :; fi > > If you run `make install` without destdir then it will install to machine's > global path depending on prefix.(If this is the case then their is genuine > "permission denied" error in the machine I think. > > If we are packaging or installing to custom target, we should pass DESTDIR. > > DESTDIR=/build/install make install > > > > Niels > > > regards > Aravinda > > On 05/12/2016 12:29 PM, Niels de Vos wrote: > > On Thu, May 12, 2016 at 01:14:07AM -0400, Raghavendra Gowdappa wrote: > > https://build.gluster.org/job/smoke/27674/console > > 06:09:06 /bin/mkdir: cannot create directory > `/usr/lib/python2.6/site-packages/gluster': Permission denied > 06:09:06 make[6]: *** [install-pyglupyPYTHON] Error 1 > > This definitely is a bug in the installation of glupy. Nothing should > get installed under /usr, teh installation process is instructed to do > its install under /build/install. > > Did someone file a bug for this yet? > > Thanks, > Niels > > 06:09:06 make[5]: *** [install-am] Error 2 > 06:09:06 make[4]: *** [install-recursive] Error 1 > 06:09:06 make[3]: *** [install-recursive] Error 1 > 06:09:06 make[2]: *** [install-recursive] Error 1 > 06:09:06 make[1]: *** [install-recursive] Error 1 > 06:09:06 make: *** [install-recursive] Error 1 > 06:09:06 Build step 'Execute shell' marked build as failure > 06:09:06 Finished: FAILURE > > regards, > Raghavendra > _______________________________________________ > Gluster-devel mailing list > Gluster-devel@xxxxxxxxxxx > http://www.gluster.org/mailman/listinfo/gluster-devel > > > _______________________________________________ > Gluster-devel mailing list > Gluster-devel@xxxxxxxxxxx > http://www.gluster.org/mailman/listinfo/gluster-devel > > > > _______________________________________________ > Gluster-devel mailing list > Gluster-devel@xxxxxxxxxxx > http://www.gluster.org/mailman/listinfo/gluster-devel _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel