public inbox for sid@sourceware.org
 help / color / mirror / Atom feed
From: fche@redhat.com (Frank Ch. Eigler)
To: bje@redhat.com (Ben Elliston)
Cc: sid@sources.redhat.com
Subject: Re: sections in configrun-sid
Date: Fri, 10 Aug 2001 01:34:00 -0000	[thread overview]
Message-ID: <o5r8ukxqhv.fsf@touchme.toronto.redhat.com> (raw)
In-Reply-To: <15219.36057.453159.217729.cygnus.project.sid@walkabout.brisbane.redhat.com>

bje wrote:

: This Perl script emits chunks of configuration data to three ordered
: sections (named "$first_section", etc).  [...]
: I am finding this arrangement to be too restrictive.  Currently, I
: need to have a "$fourth_section" that makes use of some component
: instantiation done late in the third section.

Why are you instantiating components in the third section?

: Instead, I think the logic could be simplified by creating a number of
: sections, corresponding to the usual arrangement of sid configuration
: files:
: 
: 	$preamble_section
: 	$load_section
: 	$new_section
: 	$set_section
: 	$connect_section
: 	$postamble_section

I don't see how this simplifies (program) logic.  It does give better
name to the sections, but perhaps overcodifies them.  (There is no
need to cluster all new's, etc., together, as long as there is a
partial order.)


mrg wrote:

: one other thing i've been meaning to do is work out how to eliminate
: unnecessary shlibs the big list of shlibs in the generated files.

The problem is that the perl script has no easy way of knowing which
component types come from which component libraries.  You can
hard-code the knowledge, analogously to how the "tcl-event-consumer"
component is conditionally instantiated, but I'm not sure that's an
improvement.

: that should improve exectution startup times (at least.)

(Maybe, but only for the dynamically linked case.)


- FChE

       reply	other threads:[~2001-08-10  1:34 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <15219.36057.453159.217729.cygnus.project.sid@walkabout.brisbane.redhat.com>
2001-08-10  1:34 ` Frank Ch. Eigler [this message]
2001-08-10  0:27 Ben Elliston
2001-08-10  0:58 ` matthew green

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=o5r8ukxqhv.fsf@touchme.toronto.redhat.com \
    --to=fche@redhat.com \
    --cc=bje@redhat.com \
    --cc=sid@sources.redhat.com \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).