On 3/18/21 6:21 PM, Scott Andrews wrote:
On Thu, 18 Mar 2021 11:51:09 -0500
Bruce Dubbs <bruce.du...@gmail.com> wrote:

On 3/18/21 11:21 AM, Scott Andrews wrote:

I am presently looking at and working on the LFS boot scripts.
They are in my opinion very rough state.

I am going to clean them up and use the following format for all
of the individual scripts that will be used on my systems as
follows:

Shebang line:           #!/bin/bash
Comment Title block:    the purpose of the script
Global variables:       defined here
Local variables:        not defined in functions
Source scripts:         all outside scripts sourced here
Functions:              all functions defined here
Mainline:               main body
Cleanup:                any cleanup that needs to be done
Exit:                   script terminates

I will add comments to explain the goal/purpose of the script and
also what each function does. I am going to rewrite some of the
scripts so they will be self sufficient as possible.

I will add ipv6 support using a service file much like
ivp4-static. This should allow ipv6 to be used on both dual
stacked systems and system that are ipv4 or ipv6 only.

I am also going to build to test them before placing them onto my
working servers.

Is there any interest by LFS on doing this or am I just wasting my
time posting here?

What you propose seems to be mostly documentation, but other
changes may be appropriate.  I suggest you do a couple and let us
review them. Then, with constructive feedback, the rest of the LFS
scripts.


No it will be a big cleanup and some documentation.  There are global
variables that are in scripts that have nothing to do with the script
they are in and only used in a single other script.  One of them is
used in a function and their is zero reason for it to be in
init-functions.  It is also used improperly in the other script which
most likely lead to someone putting it into init-functions to fix a
problem that should have been fixed in the other script.

I'll note though that you are the only one giving feedback and the
scripts have only had minor changes since at least 2011.

I had no reason to look at the scripts before hand until now.  I
started on a project to add ipv6, so I reviewed the boot scripts to
add the new "service" and was horrified by what I saw.  Before
reviewing the scripts my intent was to just add a "service" script,
but upon the research I did to accomplish just that I saw that I
would have to rewrite and/or correct them.  If I was going to do all
that work I might as well rewrite them into some kind of a standard
format and then add the documentation that has bee missing since
before 2011.

The boot scripts are in need of a massive cleanup to say the least.


I think it would be a bit more productive if you refrained from slagging off other people's work. Saying you could make some improvements to the stuff would suffice, as opposed to phrases like "was horrified by what I saw" and "massive cleanup to say the least".

If you want to help, that's great, but if you just want to insult other people, then...

Bye
Tim




--
decentral.ch - IT Stuff
Tim Tassonis
Hohlstrasse 400
c/o Baubüro Insitu
8048 Zürich

st...@decentral.ch
+41 79 229 36 17
--
http://lists.linuxfromscratch.org/listinfo/lfs-support
FAQ: http://www.linuxfromscratch.org/blfs/faq.html
Unsubscribe: See the above information page

Do not top post on this list.

A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

http://en.wikipedia.org/wiki/Posting_style

Reply via email to