Hello DJ,

On 06/10/2019 01:14 AM, DJ Lucas via blfs-dev wrote:

On 6/9/2019 3:45 PM, Jean-Marc Pigeon via blfs-dev wrote:
Hello,

/sbin/ifup was changed between lfs-bootscripts-20180820
and fs-bootscripts-20190524

Sequence
;-------------------
# Bring up the interface
if [ "$VIRTINT" != "yes" ]; then
   up ${IFACE}
fi

for S in ${SERVICE}; do
;-------------------

This cause trouble in case you have defined a bridge interface.
(a bridge interface is now a virtual one?),
as the bridge can NOT be UP before service bridge define it.

There is a kind of discrepancy between
line 99:
if [ "$VIRTINT" != "yes" ]; then
   up ${IFACE}

and line 122:
if [ "${VIRTINT}" == "yes" ]; then
   up ${IFACE}


Comments??

This was answered in a private thread, but repeating here in case it
comes up again. For any virtual interfaces, you need to add
"VIRTINT=yes" to the configuration. This is to avoid a couple of
problems with the earlier incarnations of this script (and Beyond BLFS
scripts). The order of operations has changed a bit to avoid the
unnecessary boot_mesg*2 cosmetic entries and disposing of the bad return
value, as well as bringing up a virtual interface before it is
configured by the service (and to fix a couple of other odd corner cases).

HTH

Let me comments on this, trying not to be mean but rather constructive.
(after all, the whole LFS project purpose is to learn).

Lets be blunt and summarize your previous statement as I understand it.

"previous script incarnation are messy, I have problems with message cosmetic entries, such lets add a a variable (VIRTINT) to configuration STANDARD to help me write code).
Then in the 20190524 iteration,  ifup code become such:

# If interface ISNOT virtual then configure interface
# prepare interface
# if interface IS virtual then configure interface.

May I suggest (as it was previously)

# prepare interface
# configure interface

Procedure "prepare interface" should hide and take
care about real/virtual interface context (layering).
Adding/extending standard or adding complexity to
(current) configuration files is always tricky.
My 2 cents.


On a more philosophical stand, "we" have a big problem
with Linux components layering. Very hard to predict
a distribution behaviour, it all depends about
compilation context and sequence (IMHO, This is LFS
project interest, trying to address this...)

For each of us, the problem is not to write linesssss
of code, the problem is to write lean and clean code.
(This is the very real challenge, ask to Boeing Max 9 team :( )

--
seen "Linux from scratch" and looking for ISO files
www.osukiss.org

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature

-- 
http://lists.linuxfromscratch.org/listinfo/blfs-dev
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Reply via email to