ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Sergey Vlasov <vsu@altlinux.ru>
To: ALT Devel discussion list <devel@lists.altlinux.org>
Subject: Re: [devel] Q: sysfsutils
Date: Tue, 30 Jan 2007 18:37:00 +0300
Message-ID: <20070130153700.GE3375@master.mivlgu.local> (raw)
In-Reply-To: <20070130151203.GW15596@immo.ru>

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

On Tue, Jan 30, 2007 at 06:12:03PM +0300, Alexey I. Froloff wrote:
> * Valery V. Inozemtsev <shrek@> [070130 17:43]:
> > > Там этого барахла вагон и маленькая тележка.  Начиная с dlist_*.
> > > Я ниасилю...
> > "пилите, Шура, она золотая" (с)
> > короче, назвался груздем...
> Т.е. предложение ваше, сделать version script примерно следующего
> вида:
> 
> SYSFS_BASE {
> 	local:
> 		dlist_*;

Насколько я понимаю, функции dlist_* всё-таки относятся к API libsysfs,
поэтому скрывать их не нужно.

> 		add_attribute;
> 		get_attribute;
> 		get_attributes_list;
> 		get_dev_attributes_list;
> 		read_dir_links;
> 		read_dir_subdirs;
> 		_*;

Так сделать тоже не получится, поскольку есть функции _dlist_*, которые
помечены как внутренние, но на самом деле используются в макросах из
dlist.h.  Хотя вообще странно - в этом dlist.h таких функций объявлено
довольно много, а используется таким образом только _dlist_mark_move.

> }

SYSFS_BASE тут можно было не заводить - local: можно положить в любую
версию.
> 
> SYSFS_2.1.0 {
> 	global:
> 		sysfs_close_dev_tree;
> 		sysfs_open_device_tree;
> 		sysfs_open_link_list;
> 		sysfs_read_dir_subdirs;
> };
> 
> ?
> 
> По мотивам
> http://lists.altlinux.org/pipermail/devel/2005-September/023881.html

Вообще этот способ чреват багами, если апстрим не использует соглашения
типа "имена вида _* предназначены для внутренних функций, которые не
должны экспортироваться из библиотеки".

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

  reply	other threads:[~2007-01-30 15:37 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-30 13:00 Alexey I. Froloff
2007-01-30 13:31 ` Anton Farygin
2007-01-30 13:36 ` Alexey I. Froloff
2007-01-30 13:56   ` Anton Farygin
2007-01-30 14:07   ` Sergey Vlasov
2007-01-30 14:38     ` Alexey I. Froloff
2007-01-30 14:42       ` Valery V. Inozemtsev
2007-01-30 15:12         ` Alexey I. Froloff
2007-01-30 15:37           ` Sergey Vlasov [this message]
2007-01-30 16:43             ` Valery V. Inozemtsev
2007-01-30 21:44         ` Alexey I. Froloff
2007-01-31  9:00           ` Anton Farygin

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=20070130153700.GE3375@master.mivlgu.local \
    --to=vsu@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