[Gbootroot-devel] Re: gBootRoot - CVS 1.20

[Jonathan R. <mtt...@ac...>]

On Sun, 3 Sep 2000, Cristian Ionescu-Idbohrn wrote:

> On Sat, 2 Sep 2000, Jonathan Rosenbaum wrote:
> 
> > On Sun, 3 Sep 2000, Cristian Ionescu-Idbohrn wrote:
> > 
> > > 
> > > Did a quick test with 1.20; found 2 bugs which I reported to the bug
> > > list.
> > 
> > O.k., these two bugs are fixed.  Thanks for observing them.  For personal
> > use, I think filling in append options in both bootdisk and normalboot is
> > acceptable.  For other people's computers, they will have to know the
> > append options for their own system configurations anyways.
> 
> OK.
> 
> > > It looks good, otherwise.
> > > 
> > > Do you think it may be possible to make the text fields in the advanced
> > > section a bit wider?
> > 
> > I added this as an "program enhancement" bug.  Close the bug if you like
> > the changes I made.  Do you want all the fields wider? 
> 
> Changes OK. I want to close the bug but I don't seem to find any
> appropriate command for that. I might not have the permition to do that.

Just choose "Fixed" or "Works For Me" from "Resolution:" and "Closed" from
"Status:", and than press the "Submit Changes" button at the bottom of the
page.  This is bug 113466.
 
> > Realize you can use emac commands in these
> > fields, pretty cool, eh?
> 
> No, I don't. Can you be more specific.

With Gtk you can do all this stuff.   Maybe I should add this to the FAQ?
 
Motion Shortcuts 

      Ctrl-A Beginning of line 
      Ctrl-E End of line 
      Ctrl-N Next Line 
      Ctrl-P Previous Line 
      Ctrl-B Backward one character 
      Ctrl-F Forward one character 
      Alt-B Backward one word 
      Alt-F Forward one word 

Editing Shortcuts 

      Ctrl-H Delete Backward Character (Backspace) 
      Ctrl-D Delete Forward Character (Delete) 
      Ctrl-W Delete Backward Word 
      Alt-D Delete Forward Word 
      Ctrl-K Delete to end of line 
      Ctrl-U Delete line 

Selection Shortcuts 

      Ctrl-X Cut to clipboard 
      Ctrl-C Copy to clipboard 
      Ctrl-V Paste from clipboard 

> Made a hash out of @entry_advanced. Documents cleanly what is what. 
> Much easier to maintain too.  I also seemed to have hit a bug in sub
> entry_advanced. It was because I was trying to use $_[index] directly. 
> Kept getting "Use of uninitialized value at ... line ...". I thought the
> reason was the fact I don't grok PerlTk, or I made some stupid
>  mistake elsewhere.
> Using local variables for the in-params put me back on the right track
> (see more below).

Add this as a program enhancement, too.  At this point, with the stable
release coming very soon it may introduce bugs, but it is an interesting
approach. 

> @container is probably a good candidate for a hash too.

I wouldn't fool with this array at this point, it's a major part of
submit(), see previous statement.

> Would be much easier to follow what the subs are doing if they
> documented the in-params (check out my comment in sub entry_advanced).

Submit this as a program enhancement bug - it's low priority right now.  I
was just using programming shorthand.  But, you are right, it would
definitely help people understand what is going on. 

> When I have both scalar, array and subroutine with the same name,
> the thing I do is try to make it obvious which is which by
> prepending the sub with a &. But that's, again, that sort of religion
> I'm suffering of ;-)

Yes, this is programming style.  When Perl4 was used that was pretty
standard, but sub() seems to be more popular now.  That's interesting ..
&sub(). :)
 
> The HELP needs some header update (date, version and the like).
> Should also describe the advanced options.

I just did this before you wrote, but I'll apply your cleaned-up look
approach.  I'll submit a proposed version number for the next Stable
Release in the next CVS, too.  

Check out the changes on the web page at the.netpedia.net/gBootRoot.html.
I will probably be moving everying to gbootroot.sourceforge.net before the
next release.

>I think the default floppy density should be set to 1440. Advanced
>users may want to fiddle with it.
>I always used the /dev/fd0u1722 floppy device when dealing with the
>1722 desity. But I now realized the generic /dev/fd0 can probably be
>used for most things, except formatting. By the way, shouldn't there
>be some words mentioned abou formatting the floppies?

Let's try out the present tomsrtbt optimized default approach in the
stable release and see what happens.  Advanced users can make the changes
anyways.  Ext2fs takes care of the formatting issues, basically anything
that's on the floppy is history.  That's why I have lots of checks for
this.