ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Pavlov Konstantin <thresh@altlinux.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] device mapper version script
Date: Sat, 16 Sep 2006 02:12:19 +0400
Message-ID: <200609160212.22146.thresh@altlinux.ru> (raw)
In-Reply-To: <20060915172844.GA10691@procyon.home>

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

On Friday 15 September 2006 21:28, Sergey Vlasov wrote:

> В окончательном пакете это делать не нужно - я предложил сделать это
> временно, чтобы пакет хоть как-то собрался, и к нему можно было бы
> применить утилиты для сравнения набора экспортируемых символов.

А, OK.

> >                 *;
> >  };
> >
> > @@ -116,3 +115,27 @@
>
> Где-то в промежутке должно было бы быть удаление dm_strdup (хотя, как
> оказалось, несуществующие символы ld просто игнорирует).
>
> Вообще история с dm_strdup следующая:
>
> http://www.redhat.com/archives/dm-devel/2006-February/msg00057.html
>
> Т.е., upstream сознательно разломал ABI.

Да, это тоже понятно.

> >                 dm_tree_preload_children;
> >                 dm_tree_suspend_children;
> >  };
> > +
> > +DEVMAPPER_1.02.09 {
> > +    global:
> > +        _align_chunk;
> > +        _fini;
> > +        _init;
> > +        _new_chunk;
> > +        add_dev_node;
> > +        create_dir;
> > +        create_target;
> > +        dm_check_version;
> > +        dm_log;
> > +        dm_segtypes;
> > +        dm_strdup_aux;
> > +        dm_task_set_geometry;
> > +        dm_task_set_gid;
> > +        dm_task_set_mode;
> > +        dm_task_set_uid;
> > +        dm_tree_skip_lockfs;
> > +        rename_dev_node;
> > +        rm_dev_node;
> > +        update_devs;
> > +};
>
> Здесь два замечания:
>
> 1) Далеко не все эти символы нужно экспортировать - скорее всего,
>    здесь должны быть только символы вида dm_*.  Более того, в
>    оригинальных исходниках уже есть файл .exported_symbols, который
>    применяется для явного указания экспортируемых символов (правда,
>    там всё загоняется в одну версию с именем Base).  Очевидно, что в
>    списке символов с версиями должны быть те и только те символы,
>    которые перечислены в файле .exported_symbols (возможно, это даже
>    не все символы вида dm_*).
>
> 2) В version script можно ещё указывать зависимости между версиями -
>    написать, что новая версия базируется на предыдущей.  Хотя я не
>    совсем понимаю смысл этого действия для варианта GNU - всё равно
>    переносить символы из одной версии в другую нельзя; похоже, это
>    было нужно в основном для исходного варианта от Sun.
>
>    Впрочем, в существующем файле для версии 1.02.02 эти зависимости
>    уже отсутствуют - не уверен, что от их добавления ничего не
>    сломается.

Понятно, принял к сведению.

On Saturday 16 September 2006 00:48, Dmitry V. Levin wrote:
> Вопрос немного в сторону: что, upstream совсем невменяем, что не в
> состоянии самостоятельно поддерживать обратную совместимость и versioning?
> Вроде бы и Drepper недалеко...

Это я проверю. Собственно, у меня почему-то есть предубеждение, что не-мой 
пакет запакован "правильно". Хотя пора бы уже понять обратное. :)

> В таком виде этот скрипт непригоден для публикации пакета.
>
> [...]
>
> > Багу (ну, enhancement) на device-mapper я уже повесил (#10006).
>
> Будет хорошо, если сперва вы его доработаете.

Обязательно, для того сюда и написал.

(Это мое первое столкнование с подобного рода вещами).

Спасибо.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

  parent reply	other threads:[~2006-09-15 22:12 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-15 15:55 [devel] lvm2 + device-mapper хочу обновить Pavlov Konstantin
2006-09-15 16:06 ` [devel] device mapper version script Pavlov Konstantin
2006-09-15 17:28   ` Sergey Vlasov
2006-09-15 20:48     ` Dmitry V. Levin
2006-09-15 22:12     ` Pavlov Konstantin [this message]
2006-09-15 22:31       ` Dmitry V. Levin
2006-09-15 22:52         ` Pavlov Konstantin
2006-09-15 23:00           ` Dmitry V. Levin
2006-09-16 15:19         ` Sergey Vlasov
2006-10-19 14:46           ` Pavlov Konstantin
2006-09-15 22:24     ` Dmitry V. Levin
2006-09-15 20:46 ` [devel] lvm2 + device-mapper хочу обновить Dmitry V. Levin

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=200609160212.22146.thresh@altlinux.ru \
    --to=thresh@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