Honor configure --prefix option.

The --prefix option was entirely ignored and DESTDIR was a combination of DESTDIR and bindir.

Bring both in line with recommendations for autoconf and make as specified in https://www.gnu.org/software/make/manual/html_node/Directory-Variables.html and https://www.gnu.org/prep/standards/html_node/DESTDIR.html.

Suggested by Daniel Westermann.

doc/xml/release.xml

@@ -47,6 +47,14 @@
                         <p>Remove hard-coded <postgres/> user so <id>$PGUSER</id> works.</p>
                     </release-item>
 
+                    <release-item>
+                        <release-item-contributor-list>
+                            <release-item-ideator id="daniel.westermann"/>
+                        </release-item-contributor-list>
+
+                        <p>Honor <proper>configure</proper> <setting>--prefix</setting> option.</p>
+                    </release-item>
+
                     <release-item>
                         <p>Rename <br-option>repo-s3-verify-ssl</br-option> option to <br-option>repo-s3-verify-tls</br-option>.</p>
 
@@ -7160,6 +7168,11 @@
             <contributor-id type="github">farrellit</contributor-id>
         </contributor>
 
+        <contributor id="daniel.westermann">
+            <contributor-name-display>Daniel Westermann</contributor-name-display>
+            <contributor-id type="github">danielwestermann</contributor-id>
+        </contributor>
+
         <contributor id="david.youatt">
             <contributor-name-display>David Youatt</contributor-name-display>
             <contributor-id type="github">youattd</contributor-id>

src/Makefile.in

@@ -34,10 +34,11 @@ LDLIBRARY = @LIBS@
 LDMAKE = $(LDLIBRARY)
 
 ####################################################################################################################################
-# Install options
+# Directory options
 ####################################################################################################################################
-# Modify destination install directory
-DESTDIR = /usr/bin
+prefix = @prefix@
+exec_prefix = @exec_prefix@
+bindir = @bindir@
 
 ####################################################################################################################################
 # List of required source files.  main.c should always be listed last and the rest in alpha order.
@@ -171,14 +172,14 @@ pgbackrest: $(OBJS)
 # Installation.  DESTDIR can be used to modify the install location.
 ####################################################################################################################################
 install: pgbackrest
-	install -d $(DESTDIR)
-	install -m 755 pgbackrest $(DESTDIR)
+	install -d $(DESTDIR)$(bindir)
+	install -m 755 pgbackrest $(DESTDIR)$(bindir)
 
 ####################################################################################################################################
 # Uninstallation.  DESTDIR should be set to the same value as when installed.
 ####################################################################################################################################
 uninstall:
-	rm -f $(DESTDIR)/pgbackrest
+	rm -f $(DESTDIR)$(bindir)/pgbackrest
 
 ####################################################################################################################################
 # Clean object files and executable created by make

test/patch/debian-package.patch

@@ -0,0 +1,20 @@
+--- rules
++++ rules
+@@ -13,7 +13,7 @@
+ 
+ override_dh_auto_configure:
+ 	# src contains a seperate configure script.
+-	cd $(CURDIR)/src && ./configure
++	cd $(CURDIR)/src && ./configure --prefix=/usr
+ 	dh_auto_configure
+ 
+ override_dh_auto_build:
+@@ -36,7 +36,7 @@
+ 	dh_auto_clean
+ 
+ override_dh_auto_install:
+-	make -C src install DESTDIR=$(CURDIR)/debian/pgbackrest/usr/bin
++	make -C src install DESTDIR=$(CURDIR)/debian/pgbackrest
+ 	dh_auto_install
+ 
+ .PHONY: build