ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Mikhail Efremov <sem@altlinux.ru>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] нужна консультация по autotools (mc 4.8.6)
Date: Sat, 22 Sep 2012 22:39:59 +0400
Message-ID: <20120922223959.3ae483b2@sem-notebook> (raw)
In-Reply-To: <201209222207.46999.asy@altlinux.ru>

On Sat, 22 Sep 2012 22:07:46 +0400 Sergey Y. Afonin wrote:
> On Saturday 22 September 2012, Mikhail Efremov wrote:
> 
> > Прежде чем подозревать ошибку в Makefile нужно убедиться, что
> > DESTDIR действительно определяется при make install. Есть
> > %makeinstall_std, кстати.
> 
> Так 4.7.5.6 собирается, а там аналогичная конструкция тоже
> встречается. Да и в 4.8.6 есть 
> 
> install-man1: $(man_MANS)
>         @$(NORMAL_INSTALL)
>         test -z "$(man1dir)" || $(MKDIR_P) "$(DESTDIR)$(man1dir)"
> 
> И оно работает:
> 
> Making install in man
> make[2]: Entering directory `/home/asy/RPM/BUILD/mc-4.8.6/doc/man'
> Making install in es
> make[3]: Entering directory `/home/asy/RPM/BUILD/mc-4.8.6/doc/man/es'
> make[4]: Entering directory `/home/asy/RPM/BUILD/mc-4.8.6/doc/man/es'
> make[4]: Nothing to be done for `install-exec-am'.
> test -z "/home/asy/tmp-build/mc-buildroot/usr/share/man/man1"
> || /bin/mkdir -p "/home/asy/tmp-build/mc-buildroot/usr/share/man/man1"
> 
> То есть, тут DESTDIR получается определён. А попытка сделать как-то
> так %makeinstall DESTDIR=$RPM_BUILD_ROOT,  предсказуемо приводит к
> помещению основного набора файлов в $RPM_BUILD_ROOT$RPM_BUILD_ROOT.
> Можно, конечно, вытащить всё это в правильное место и собрать пакет,
> но это как-то неправильно.

%makeinstall не определяет DESTDIR. Зато переопределяет mandir и еще
кучу переменных куда-то в %buildroot. Поэтому при одновременном
определении DESTDIR в %buildroot и получается %buildroot%buildroot.
Используйте %makeinstall_std.

-- 
WBR, Mikhail Efremov


  reply	other threads:[~2012-09-22 18:39 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-22 13:25 Sergey Y. Afonin
2012-09-22 13:59 ` Mikhail Efremov
2012-09-22 18:07   ` Sergey Y. Afonin
2012-09-22 18:39     ` Mikhail Efremov [this message]
2012-09-22 18:58       ` Sergey Y. Afonin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120922223959.3ae483b2@sem-notebook \
    --to=sem@altlinux.ru \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git