ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Valery V. Inozemtsev" <shrek@altlinux.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] chkfontpath
Date: Thu, 30 Aug 2007 20:52:38 +0400
Message-ID: <200708302052.38608.shrek@altlinux.ru> (raw)
In-Reply-To: <20070830164427.GE10845@basalt.office.altlinux.org>

[-- Attachment #1: Type: text/plain, Size: 2259 bytes --]

В сообщении от 30 августа 2007 Dmitry V. Levin написал(a):
> On Thu, Aug 30, 2007 at 08:23:19PM +0400, Valery V. Inozemtsev wrote:
> [...]
>
> > %install
> > %__mkdir_p %buildroot%_fontsdir/%fname
> > %__install -m644 *.ttf %buildroot%_fontsdir/%fname
>
> Почему не просто "mkdir -p" и "install"?
>
> > %_bindir/mkfontscale %buildroot%_fontsdir/%fname
>
> Зачем %_bindir?
>
> > %__ln_s fonts.scale %buildroot%_fontsdir/%fname/fonts.dir
>
> Почему не просто "ln -s"?
>
> > %__mkdir_p %buildroot%_sysconfdir/X11/fontpath.d
> > %__ln_s ../../..%_fontsdir/%fname
> > %buildroot%_sysconfdir/X11/fontpath.d/ttf-%fname:pri=50
>
> Аналогичные вопросы.

не знаю, привычки. ну раз уж я сам объявил макросы злом, будем от них 
избавляться

>
> > имена симлинков в /etc/X11/fontpath.d следующие:
> > для ttf шрифтов ttf-%name:pri=50
> > для otf - otf-%name:pri=50
> > для type1 - type1-%name:pri=40
> > для bitmap - bitmap-%name:pri=20 если шрифты не зависят от локали и
> > bitmap-%name:pri=10 для зависимых от локали.
> > ":pri=" это порядок в котором шрифты будет выбираться
>
> Ой как сложно.  Может завести пачку макросов, например,
> %make_fontpath_ttf(%fname,50) или что-нибудь ещё более простое?
>
> > далее триггер (только для пакетов у которых не меняется имя, изменяется
> > только %version или %release): %triggerin -- %name <= 0.2-alt1
>
> Почему %triggerin??
> Тогда уж %triggerun или %triggerpostun

ну если ты настаиваешь пусть будет %triggerun

>
> > [ ! -x %_sbindir/chkfontpath -a ! -f %_sysconfdir/X11/fs/config ] || \
> >         %_sbindir/chkfontpath -q -r %_fontsdir/%fname ||:
>
> Условие написано с ошибкой.

да, сам только заметил.

> Лучше уж простой, но правильный if: 
> if [ -x %_sbindir/chkfontpath -a -f %_sysconfdir/X11/fs/config ]; then
> 	%_sbindir/chkfontpath -q -r %_fontsdir/%fname ||:
> fi

угу

>
> > %_sysconfdir/X11/fontpath.d/*
>
> Каталог %_sysconfdir/X11/fontpath.d/ тоже хорошо бы замакрить.
>
> > что тут заворачивать в макросы мне не понятно. спек для примера во
> > вложении
>
> 1. каталог %_sysconfdir/X11/fontpath.d;

ну это само собой

> 2. команда на создание ссылки в %buildroot%_sysconfdir/X11/fontpath.d/;

сомнительно

> 3. весь триггер от начала и до конца.

а если каталогов несколько?

-- 
Valery V. Inozemtsev

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

  reply	other threads:[~2007-08-30 16:52 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-28  7:56 [devel] libXfont >= 1.2.9 - прощай chkfontpath Valery V. Inozemtsev
2007-08-28 10:57 ` Valery V. Inozemtsev
2007-08-28 17:01   ` Denis Medvedev
2007-08-28 18:10     ` Valery V. Inozemtsev
2007-08-29  5:26 ` [devel] да здравствует chkfontpath для /etc/X11/fontpath.d Vitaly Lipatov
2007-08-29  5:56   ` Valery V. Inozemtsev
2007-08-29  7:23     ` Alexey Rusakov
2007-08-29 10:15       ` Valery V. Inozemtsev
2007-08-29 13:13         ` Alexey Rusakov
2007-08-29 13:27           ` Valery V. Inozemtsev
2007-08-29 14:41             ` Alexey Rusakov
2007-08-29 17:40               ` Valery V. Inozemtsev
2007-08-29 20:01             ` Vitaly Lipatov
2007-08-30  5:44               ` Valery V. Inozemtsev
2007-08-30  8:23                 ` [devel] chkfontpath Alexey Rusakov
2007-08-30  8:46                   ` Alexey Rusakov
2007-08-30 16:23                     ` Valery V. Inozemtsev
2007-08-30 16:29                       ` Dmitry V. Levin
2007-08-30 16:35                         ` Valery V. Inozemtsev
2007-08-30 16:44                       ` Dmitry V. Levin
2007-08-30 16:52                         ` Valery V. Inozemtsev [this message]
2007-08-30 16:44                       ` Alexey I. Froloff
2007-08-30 17:17                         ` Valery V. Inozemtsev
2007-08-30 17:21                           ` Dmitry V. Levin
2007-08-30 17:27                             ` Valery V. Inozemtsev
2007-08-30 17:34                               ` Dmitry V. Levin
2007-08-31  5:38                                 ` Valery V. Inozemtsev
2007-08-31  9:34                                   ` Dmitry V. Levin
2007-08-31  9:49                                     ` Valery V. Inozemtsev
2007-08-30 17:10                       ` Valery V. Inozemtsev
2007-08-31  8:57                       ` Хихин Руслан
2007-08-31  9:08                         ` Valery V. Inozemtsev
2007-08-31 17:53                           ` Хихин Руслан
2007-08-31 18:27                             ` Valery V. Inozemtsev
2007-08-31 18:11                           ` Хихин Руслан
2007-08-31 19:07                             ` Valery V. Inozemtsev
2007-08-31 19:33                               ` Хихин Руслан
2007-08-29  9:37     ` [devel] да здравствует chkfontpath для /etc/X11/fontpath.d Slava Semushin

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=200708302052.38608.shrek@altlinux.ru \
    --to=shrek@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