Devicetree
 help / color / mirror / Atom feed
From: Josh Boyer <jwboyer-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
To: David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>
Cc: devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.org
Subject: Re: [PATCH 1/2] libfdt: Add version.lds file
Date: Tue, 25 Nov 2008 20:57:18 -0500	[thread overview]
Message-ID: <20081125205718.68a92b73@zod.rchland.ibm.com> (raw)
In-Reply-To: <20081126014753.GF3420-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>

On Wed, 26 Nov 2008 12:47:53 +1100
David Gibson <david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org> wrote:

> > The hard-coded list is pretty simple, at the risk of missing things.
> 
> I'm actually not too wirried about missing things.  If we miss
> something, it's pretty easy to fix.  It probably would be a good idea
> to make the testsuite build against the shared version of the library
> to verify that at least the functions exercised there are correctly
> exposed.

That's a good idea.  I'll see what I can do there.

> I'm marginally more worried about an overbroad wildcard including
> something we didn't really want exposed.  I think the versioning
> script needs a:
> 	local:
> 		*
> after the global list to to inhibit exposing everything else.

Ah, yes.  It would be good for someone with more of a clue than me to
vet the list I have in the file right now, and make sure I didn't
include/omit things that shouldn't/should be there.  Once that is done,
I can do the local thing.

josh

  parent reply	other threads:[~2008-11-26  1:57 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-25 14:21 [PATCH 1/2] libfdt: Add version.lds file Josh Boyer
     [not found] ` <20081125142149.GB2340-JVa43G0qJ78RlTYtMZUE4cJ8OCA1ciLf@public.gmane.org>
2008-11-25 16:50   ` Jon Loeliger
     [not found]     ` <E1L517Q-0001Is-Fh-CYoMK+44s/E@public.gmane.org>
2008-11-25 16:57       ` Josh Boyer
     [not found]         ` <20081125165747.GB17258-JVa43G0qJ78RlTYtMZUE4cJ8OCA1ciLf@public.gmane.org>
2008-11-25 17:16           ` Josh Boyer
     [not found]             ` <20081125171648.GC17258-JVa43G0qJ78RlTYtMZUE4cJ8OCA1ciLf@public.gmane.org>
2008-11-26  1:47               ` David Gibson
     [not found]                 ` <20081126014753.GF3420-787xzQ0H9iRg7VrjXcPTGA@public.gmane.org>
2008-11-26  1:57                   ` Josh Boyer [this message]
     [not found]                     ` <20081125205718.68a92b73-jnIq6iUNB1CO8iYQgJmm/0EOCMrvLtNR@public.gmane.org>
2008-11-26  2:11                       ` David Gibson

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=20081125205718.68a92b73@zod.rchland.ibm.com \
    --to=jwboyer-23vcf4htsmix0ybbhkvfkdbpr1lh4cv8@public.gmane.org \
    --cc=david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org \
    --cc=devicetree-discuss-mnsaURCQ41sdnm+yROfE0A@public.gmane.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox