All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] Disable functionality that doesn't work on MINGW32
@ 2011-09-21 21:29 Mario Limonciello
  2011-09-21 22:02 ` Vladimir 'φ-coder/phcoder' Serbinenko
  0 siblings, 1 reply; 2+ messages in thread
From: Mario Limonciello @ 2011-09-21 21:29 UTC (permalink / raw)
  To: grub-devel@gnu.org

The following two patches are for disabling some specifics that will not work on MINGW32.

Vladimir explicitly expressed disinterest in going the route of #undef GRUB_UTIL, but it seems to be the most appropriate method in the first patch to me because MINGW32 does not support ttys and won't ever.

The second patch just matches other code, since there is no grub_util_get_fd_sectors defined on MINGW32.  This will at least get the file compiling with potentially broken functionality, which I see as an improvement over not compiling and no functionality.

-- 
*Mario Limonciello*
Linux Engineer
*Dell*| OS Engineering


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 0/2] Disable functionality that doesn't work on MINGW32
  2011-09-21 21:29 [PATCH 0/2] Disable functionality that doesn't work on MINGW32 Mario Limonciello
@ 2011-09-21 22:02 ` Vladimir 'φ-coder/phcoder' Serbinenko
  0 siblings, 0 replies; 2+ messages in thread
From: Vladimir 'φ-coder/phcoder' Serbinenko @ 2011-09-21 22:02 UTC (permalink / raw)
  To: grub-devel

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

On 21.09.2011 23:29, Mario Limonciello wrote:
> The following two patches are for disabling some specifics that will
> not work on MINGW32.
>
> Vladimir explicitly expressed disinterest in going the route of #undef
> GRUB_UTIL, but it seems to be the most appropriate method in the first
> patch to me because MINGW32 does not support ttys and won't ever.
>
GRUB_UTIL means that we run under an OS and use it. While Windows is a
low-quality one, it's still one, so GRUB_UTIL must be defined. Correct
way to dial with this issue is to add a windows equivalent of this code
(requesting password from user).
> The second patch just matches other code, since there is no
> grub_util_get_fd_sectors defined on MINGW32.  This will at least get
> the file compiling with potentially broken functionality, which I see
> as an improvement over not compiling and no functionality.
>


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-09-21 22:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-09-21 21:29 [PATCH 0/2] Disable functionality that doesn't work on MINGW32 Mario Limonciello
2011-09-21 22:02 ` Vladimir 'φ-coder/phcoder' Serbinenko

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.