Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Nov 2013 06:57:02 -0800
From:      Alfred Perlstein <alfred@freebsd.org>
To:        Devin Teske <dteske@freebsd.org>
Cc:        FreeBSD Hackers <freebsd-hackers@freebsd.org>, "Teske, Devin" <Devin.Teske@fisglobal.com>
Subject:   Re: Loader forth changes for customization
Message-ID:  <5283933E.30603@freebsd.org>
In-Reply-To: <09673101-DB54-4D25-9989-8C80D06E266B@fisglobal.com>
References:  <5282E56F.4020307@freebsd.org> <52832003.8080406@freebsd.org> <09673101-DB54-4D25-9989-8C80D06E266B@fisglobal.com>

next in thread | previous in thread | raw e-mail | index | archive | help

On 11/13/13, 12:18 AM, Teske, Devin wrote:
> On Nov 12, 2013, at 10:45 PM, Alfred Perlstein wrote:
>
>> I added some hooks for menu.rc as well, you can see it via the github urls below.
>>
>> I've attached a sample menu.rc.local that provides additional menus.
>>
>> -Alfred
>>
>> On 11/12/13, 6:35 PM, Alfred Perlstein wrote:
>>> Hey folks,
>>>
>>> I added some forth using Devin's help to make it easier to customize the FreeBSD boot loader graphics.
>>>
>>> Diffs are here:
>>> https://github.com/alfredperlstein/freebsd/compare/loader_custom_rc
>>> -or-
>>> https://github.com/alfredperlstein/freebsd/compare/loader_custom_rc.diff
>>>
>>>
>>> Diff attached.
>>>
>>> Also attached is a custom loader.rc file and loader.conf file that shows how to set the brand/logo.
>>>
>>> Please review.
>>>
> I signed up for a github account (thanks), and I started commenting on some lines.
yay! :)
>
>>> -Alfred
>> <menu.rc.local>
> Hmmm, I hadn't realized that you could say:
>
> set foo=bar
>
> Along-side setting functions in the same file.
>
> I don't think you can set functions in an *.rc file, only in a *.4th file?
>
> No? Maybe it's a false misconception of mine. I've been keeping them
> separate for years. (but probably rightfully so, to keep *.rc files clean).

It seems to work although I will talk to the team about making separate 
files for the set commands.

I've responded to your review comments here:

https://github.com/alfredperlstein/freebsd/commit/0ca72dccd78b880b3e3ef4c2bb9ce025950a370b#commitcomment-4584862

The changes I made are now in the branch
https://github.com/alfredperlstein/freebsd/tree/loader_custom_rc and you 
likely should see them in the updated pull request I sent you.



-Alfred



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5283933E.30603>