Re: CVS commit: src/distrib/utils/embedded

[Hisashi T Fujinaka <htodd%twofifty.com@localhost>]

On Sat, 6 Apr 2013, Izumi Tsutsui wrote:

> > Module Name:    src
> > Committed By:   htodd
> > Date:           Wed Apr  3 19:37:43 UTC 2013
> >
> > Modified Files:
> >         src/distrib/utils/embedded: mkimage
> >
> > Log Message:
> > Fix build.
>
> What build problem did you try to fix?
>
> --- mkimage     28 Mar 2013 00:20:02 -0000      1.37
> +++ mkimage     3 Apr 2013 19:37:43 -0000       1.38
> @@ -156,8 +156,8 @@ if [ -z "$bootonly" ]; then
>         echo ${bar} Populating ffs filesystem ${bar}
>         ${MAKEFS} -N ${release}/etc -t ffs -rx \
>             -O $(((${init} + ${boot} + ${swap}) / 2))m \
> -           -b $((${extra} / 2))m \
> -           -o d=4096 \
> +               -o d=4096 \
> +           -b $((${extra}))m \
>             -F "$tmp/selected_sets" ${image} "${release}" "${mnt}"
> fi
> ---
>
> If the problem is overflow of the image, shouldn't the "extra" variable
> in each config file be bumped instead of changing a unit?

Sorry for the messy commit. I just changed the -b line the way I did
because I'm not sure where extra comes from and version 1.37 broke every
img that I tried to build.

If you think there's a better way to fix it, please do.

--
Hisashi T Fujinaka - htodd%twofifty.com@localhost
BSEE(6/86) + BSChem(3/95) + BAEnglish(8/95) + MSCS(8/03) + $2.50 = latte