[PATCH] mkefidisk.sh: add deprecation warning to the output

[PATCH] mkefidisk.sh: add deprecation warning to the output

[Ed Bartosh]

mkefidisk.sh will soon be deprecated in favor of .wic images.

Added deprecation warning to the script to inform users that
this script will soon be removed from the codebase.

Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
---
 scripts/contrib/mkefidisk.sh | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
index d8db3c0..a175895 100755
--- a/scripts/contrib/mkefidisk.sh
+++ b/scripts/contrib/mkefidisk.sh
@@ -20,6 +20,11 @@
 
 LANG=C
 
+echo
+echo "WARNING: This script is deprecated and will be removed soon."
+echo "Please consider using wic EFI images instead."
+echo
+
 # Set to 1 to enable additional output
 DEBUG=0
 OUT="/dev/null"
-- 
2.1.4

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Khem Raj]

[-- Attachment #1: Type: text/plain, Size: 1241 bytes --]


> On Oct 31, 2016, at 3:46 AM, Ed Bartosh <ed.bartosh@linux.intel.com> wrote:
> 
> mkefidisk.sh will soon be deprecated in favor of .wic images.
> 
> Added deprecation warning to the script to inform users that
> this script will soon be removed from the codebase.
> 
> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> ---
> scripts/contrib/mkefidisk.sh | 5 +++++
> 1 file changed, 5 insertions(+)
> 
> diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> index d8db3c0..a175895 100755
> --- a/scripts/contrib/mkefidisk.sh
> +++ b/scripts/contrib/mkefidisk.sh
> @@ -20,6 +20,11 @@
> 
> LANG=C
> 
> +echo
> +echo "WARNING: This script is deprecated and will be removed soon."
> +echo "Please consider using wic EFI images instead."
> +echo
> +

is .wic image dd’able directly ?
We should also document, the wic steps in wiki pages e.g. http://wiki.minnowboard.org/Yocto_Project



> # Set to 1 to enable additional output
> DEBUG=0
> OUT="/dev/null"
> --
> 2.1.4
> 
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core


[-- Attachment #2: Message signed with OpenPGP using GPGMail --]
[-- Type: application/pgp-signature, Size: 211 bytes --]

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Christopher Larson]

[-- Attachment #1: Type: text/plain, Size: 1320 bytes --]

On Thu, Nov 3, 2016 at 2:42 PM, Khem Raj <raj.khem@gmail.com> wrote:

> > On Oct 31, 2016, at 3:46 AM, Ed Bartosh <ed.bartosh@linux.intel.com>
> wrote:
> >
> > mkefidisk.sh will soon be deprecated in favor of .wic images.
> >
> > Added deprecation warning to the script to inform users that
> > this script will soon be removed from the codebase.
> >
> > Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> > ---
> > scripts/contrib/mkefidisk.sh | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > index d8db3c0..a175895 100755
> > --- a/scripts/contrib/mkefidisk.sh
> > +++ b/scripts/contrib/mkefidisk.sh
> > @@ -20,6 +20,11 @@
> >
> > LANG=C
> >
> > +echo
> > +echo "WARNING: This script is deprecated and will be removed soon."
> > +echo "Please consider using wic EFI images instead."
> > +echo
> > +
>
> is .wic image dd’able directly ?
>

Yes, that’s the point :) Though I prefer bmaptool w/ IMAGE_FSTYPES +=
“wic.bmap” personally, don’t have to rely on the file keeping its
sparseness when scp’ing it around and whatnot.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 1979 bytes --]

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Ed Bartosh]

On Thu, Nov 03, 2016 at 07:23:24PM -0700, Christopher Larson wrote:
> On Thu, Nov 3, 2016 at 2:42 PM, Khem Raj <raj.khem@gmail.com> wrote:
> 
> > > On Oct 31, 2016, at 3:46 AM, Ed Bartosh <ed.bartosh@linux.intel.com>
> > wrote:
> > >
> > > mkefidisk.sh will soon be deprecated in favor of .wic images.
> > >
> > > Added deprecation warning to the script to inform users that
> > > this script will soon be removed from the codebase.
> > >
> > > Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> > > ---
> > > scripts/contrib/mkefidisk.sh | 5 +++++
> > > 1 file changed, 5 insertions(+)
> > >
> > > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > > index d8db3c0..a175895 100755
> > > --- a/scripts/contrib/mkefidisk.sh
> > > +++ b/scripts/contrib/mkefidisk.sh
> > > @@ -20,6 +20,11 @@
> > >
> > > LANG=C
> > >
> > > +echo
> > > +echo "WARNING: This script is deprecated and will be removed soon."
> > > +echo "Please consider using wic EFI images instead."
> > > +echo
> > > +
> >
> > is .wic image dd’able directly ?
> >
> 
> Yes, that’s the point :) Though I prefer bmaptool w/ IMAGE_FSTYPES +=
> “wic.bmap” personally, don’t have to rely on the file keeping its
> sparseness when scp’ing it around and whatnot.

Recommending bmaptool as a preferred way of flashing is in the plans :)

--
Regards,
Ed

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Ed Bartosh]

On Thu, Nov 03, 2016 at 02:42:39PM -0700, Khem Raj wrote:
> 
> > On Oct 31, 2016, at 3:46 AM, Ed Bartosh <ed.bartosh@linux.intel.com> wrote:
> > 
> > mkefidisk.sh will soon be deprecated in favor of .wic images.
> > 
> > Added deprecation warning to the script to inform users that
> > this script will soon be removed from the codebase.
> > 
> > Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
> > ---
> > scripts/contrib/mkefidisk.sh | 5 +++++
> > 1 file changed, 5 insertions(+)
> > 
> > diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
> > index d8db3c0..a175895 100755
> > --- a/scripts/contrib/mkefidisk.sh
> > +++ b/scripts/contrib/mkefidisk.sh
> > @@ -20,6 +20,11 @@
> > 
> > LANG=C
> > 
> > +echo
> > +echo "WARNING: This script is deprecated and will be removed soon."
> > +echo "Please consider using wic EFI images instead."
> > +echo
> > +
> 
> is .wic image dd’able directly ?
Yes, they are.

> We should also document, the wic steps in wiki pages e.g. http://wiki.minnowboard.org/Yocto_Project
It's already documented in README.hardware: https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/README.hardware

And in Yocto manual:
http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#building-an-image-for-hardware

--
Regards,
Ed

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Khem Raj]

On 11/7/16 3:00 PM, Ed Bartosh wrote:
> On Thu, Nov 03, 2016 at 02:42:39PM -0700, Khem Raj wrote:
>>
>>> On Oct 31, 2016, at 3:46 AM, Ed Bartosh <ed.bartosh@linux.intel.com> wrote:
>>>
>>> mkefidisk.sh will soon be deprecated in favor of .wic images.
>>>
>>> Added deprecation warning to the script to inform users that
>>> this script will soon be removed from the codebase.
>>>
>>> Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
>>> ---
>>> scripts/contrib/mkefidisk.sh | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/scripts/contrib/mkefidisk.sh b/scripts/contrib/mkefidisk.sh
>>> index d8db3c0..a175895 100755
>>> --- a/scripts/contrib/mkefidisk.sh
>>> +++ b/scripts/contrib/mkefidisk.sh
>>> @@ -20,6 +20,11 @@
>>>
>>> LANG=C
>>>
>>> +echo
>>> +echo "WARNING: This script is deprecated and will be removed soon."
>>> +echo "Please consider using wic EFI images instead."
>>> +echo
>>> +
>>
>> is .wic image dd’able directly ?
> Yes, they are.
> 
>> We should also document, the wic steps in wiki pages e.g. http://wiki.minnowboard.org/Yocto_Project
> It's already documented in README.hardware: https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/README.hardware
> 
> And in Yocto manual:
> http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#building-an-image-for-hardware

Thanks, now can you also nudge the minnowboard.org to do the same ?

> 
> --
> Regards,
> Ed
> 

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Randy Witt]

>>> We should also document, the wic steps in wiki pages e.g. http://wiki.minnowboard.org/Yocto_Project
>> It's already documented in README.hardware: https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/README.hardware
>>
>> And in Yocto manual:
>> http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#building-an-image-for-hardware
>
> Thanks, now can you also nudge the minnowboard.org to do the same ?
>

Pinging you John. :)

>>
>> --
>> Regards,
>> Ed
>>

Re: [PATCH] mkefidisk.sh: add deprecation warning to the output

[Ed Bartosh]

On Tue, Nov 08, 2016 at 01:05:23PM -0800, John Hawley wrote:
> On 11/08/2016 12:44 PM, Randy Witt wrote:
> >>>> We should also document, the wic steps in wiki pages e.g.
> >>>> http://wiki.minnowboard.org/Yocto_Project
> >>> It's already documented in README.hardware:
> >>> https://git.yoctoproject.org/cgit/cgit.cgi/poky/tree/README.hardware
> >>>
> >>> And in Yocto manual:
> >>> http://www.yoctoproject.org/docs/2.2/mega-manual/mega-manual.html#building-an-image-for-hardware
> >>>
> >>
> >> Thanks, now can you also nudge the minnowboard.org to do the same ?
> >>
> > 
> > Pinging you John. :)
> 
> So two obvious questions in reading the documentation off the YP site:
> 
> 1) Does WIC actually handle the EFI partition for booting off of UEFI
> systems? (guessing yes, but wanted to double check)
Yes, it does.

> 2) I assume it only creates MBR based images, seeing as the suggestion
> is to DD the image down, how does GPT based images work since you can't
> (exactly) DD a GPT image down and have it work "correctly"?
>
Please, elaborate on "correctly". Do you mean that kernel will complain
that backup GPT header is not at the end of disk? This is not handled by
wic obviously. However, even without this image is bootable and
functional.

> Sorry, I'm sure that's covered somewhere else, just want to double check
> this will be ok before I update the documentation on MB


--
Regards,
Ed