From: Alexey Tourbin <at@altlinux.ru> To: ALT Devel discussion list <devel@lists.altlinux.org> Subject: Re: [devel] libopenobex-1.3 symbol versioning patch Date: Sat, 12 Aug 2006 17:32:57 +0400 Message-ID: <20060812133257.GE24054@localhost.localdomain> (raw) In-Reply-To: <20060812130037.GC15159@procyon.home> [-- Attachment #1: Type: text/plain, Size: 3458 bytes --] On Sat, Aug 12, 2006 at 05:00:37PM +0400, Sergey Vlasov wrote: > > Насколько лучше рисковать недоэкспортом, чем переэкспортом? Вопрос > > тонкий, и я думаю, что однозначного ответа нет. При жестком подходе -- > > запрещено всё, что не разрешено -- ошибка недоэкспорта может и не > > обнаружиться, если клиент библиотеки использует AC_TRY_LINK. Более > > мягкий подход у меня ассоциируется с принципом "не навреди". А именно, > > чего мы хотим добиться? -- мы хотим добавить symbol versioning. Что > > может произойти в худшем случае? -- символы не отслеживаются и > > versioning работает лишь частично. То есть при мягком подходе мы более > > или менее добиваемся поставленной цели. А при жестком подходе в худшем > > случае мы получаем вредный side effect -- символы экспортируются не > > полностью. Это совсем не то, за что мы боролись. > > В данном случае upstream уже использует жёсткий подход - символы, не > добавленные в obex.sym, оказываются недоступными. В этом случае > применение для obex.ver мягкого подхода приведёт к возможности > появления платформозависимых ошибок - в системах, не поддерживающих > -Wl,--version-script, но при этом поддерживающих -export-symbols через > libtool, пропущенные символы будут недоступны, в то время как в > системах, поддерживающих -Wl,--version-script, такие символы останутся > доступными. Вряд ли upstream согласится на такое. Согласен. С точки зрения апстрима лучше поддерживать эксклюзивный список экспортируемых символов. Так что если пропихивать этот патч в апстрим, тогда имеет смысл синхронизировать список/карту и т.п. Но если у пакета казуальный maintainer, которому version script дали в долг, тем более если в апстриме не используется -export-symbols, тогда вряд стоит делать эту карту эксклюзивной. > Кстати, а обязан ли каждому паттерну в local соответствовать хотя бы > один реально существующий символ? Кстати, и в списке global каждому элементу в общем-то не обязан соответствовать какой-либо реально существующий символ (сюрприз!). Филькина грамота этот version script в общем. > Если нет, можно попробовать > сгенерировать что-то такого типа: > > local: > [^BFIO]*; > O[^B]*; > OB[^E]*; > OBE[^X]*; > OBEX[^_]*; > OBEX_[^....]*; > ... > OBEX_TransportDisconnec[^t]*; > OBEX_TransportDisconnect?*; > ... > > Хотя как бы с такими паттернами не напороться на баги в каком-нибудь > кривом сановском линкере (или там опции всё равно другие?), или старых > binutils. В каком смысле сгенерировать? Идея интересная, но при наличии на руках объектного материала можно генерировать не паттерны, а уже сами символы. Кстати, интересная идея мне в голову пришла: в libtool нужно добавить новую опцию: --version-script (похожую на --export-symbols). Тогда на платформах, которые не поддерживают versioning, libtool смог бы схлопывать карту verion script в список export symbols. В случае, если список эксклюзивный и все символы явно перечислены, тогда вообще всё просто и это дается чем-то вроде perl -lne '/^\s+(\w+);/&&print$1' или то же самое на sed'е, хотя перл кажется более портабельный чем sed. Но в общем виде конечно нужен парсер и работа с объектным материалом, а это с этим уже больше сложностей. Но в принципе это должно делаться где-то на уровне libtool. На то он и libtool. Либтул ведь сейчас генерирует из списка карту, что конечно же не фокус. А вот обратно ему пока слабо. [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2006-08-12 13:32 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2006-08-11 18:44 [devel] [RFC] " Sergey Vlasov 2006-08-11 19:42 ` [devel] " Alexey Tourbin 2006-08-12 10:43 ` Sergey Vlasov 2006-08-12 11:19 ` Alexey Tourbin 2006-08-12 13:00 ` Sergey Vlasov 2006-08-12 13:32 ` Alexey Tourbin [this message] 2006-08-12 13:46 ` Alexey Tourbin 2006-08-12 14:08 ` Sergey Vlasov 2006-08-15 17:19 ` Dmitry V. Levin 2006-08-12 13:50 ` [devel] [RFC] " Sergey Vlasov 2006-08-12 14:05 ` [devel] " Alexey Tourbin 2006-08-12 14:36 ` Alexey Tourbin 2006-08-12 15:49 ` Sergey Vlasov 2006-08-13 7:51 ` Alexey Tourbin 2006-08-13 12:13 ` Sergey Vlasov 2006-08-13 12:20 ` Alexey Tourbin 2006-08-13 14:55 ` Sergey Vlasov 2006-08-13 16:00 ` Alexey Tourbin 2006-08-13 18:57 ` Sergey Vlasov 2006-08-15 19:27 ` [devel] [RFC] " Sergey Vlasov
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=20060812133257.GE24054@localhost.localdomain \ --to=at@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