RTimothyEdwards / XCircuit

XCircuit circuit drawing and schematic capture tool
GNU General Public License v2.0
97 stars 25 forks source link

Configure script ignore --mandir parameter #7

Open ktrace opened 3 years ago

ktrace commented 3 years ago

Seems "mandir" parameters never used in real case and maintainer has no chance to set correct manual's location

$ grep -n -i -C2 mandir Makefile.in
189-         $(am__cd) "$$dir" && rm -f $$files; }; \
190-  }
191:man1dir = $(mandir)/man1
192-NROFF = nroff
193-MANS = $(man_MANS)
--
347-localedir = @localedir@
348-localstatedir = @localstatedir@
349:mandir = @mandir@
350-mkdir_p = @mkdir_p@
351-oldincludedir = @oldincludedir@
--
385-# Directories for app-defaults file and manual page
386-appdefaultsdir = $(libdir)/$(PACKAGE)-$(VERSION)/app-defaults
387:appmandir = $(libdir)/$(PACKAGE)-$(VERSION)/man
388-install_appdefaultsdir = @DIST_DIR@/lib/$(PACKAGE)-$(VERSION)/app-defaults
389:install_appmandir = @DIST_DIR@/lib/$(PACKAGE)-$(VERSION)/man
390-install_bindir = @DIST_DIR@/bin
391-PATHNAMES = -DPROG_VERSION=\"$(VERSION)\" \
--
918-    @echo "Installing app-defaults file"
919-    $(mkinstalldirs) $(DESTDIR)$(install_appdefaultsdir)
920:    $(mkinstalldirs) $(DESTDIR)$(install_appmandir)
921-    $(INSTALL_DATA) lib/XCircuit.ad $(DESTDIR)$(install_appdefaultsdir)/XCircuit
922:    $(INSTALL_DATA) lib/xcircuit.1 $(DESTDIR)$(install_appmandir)/xcircuit.1
923-    @echo "Installing library and netlist files"
924-    $(mkinstalldirs) $(DESTDIR)$(install_librarydir)

I think better use --mandir for installing if it set.