Bug#607417: ΑΠ: win32-loader: please offer a "Boot from network with gPXE" option

2011-02-02 Thread Alkis Georgopoulos
Στις 02-02-2011, ημέρα Τετ, και ώρα 15:09 +0100, ο/η Didier 'OdyX'
Raboud έγραψε:
> So, given the messages received so far and my present answer, I am pretty 
> confident that the PXE patch does basically what you need; I will proceed in 
> merging the pxe branch to "master". Other enhancements will certainly happen, 
> but please report separate bugs (it's not a matter of blocking discussion, 
> but 
> to keep the "one problem is one bug" motto).

Confirming everything that OdyX says. Awesome PXE integration, much more
mature than our preliminary attempts. :)

To compile, I did the following:
git clone git://git.debian.org/d-i/win32-loader.git
cd win32-loader
git checkout -b pxe remotes/origin/pxe
# remove "--format=i386-pc" from Makefile because I had an old grub version
cp /usr/share/gpxe/gpxe.lkrn ./pxe.lkrn
PXE=yes make

Tested the resulting win32-loader.exe in a vbox XP installation, worked
fine.

The only thing that remains is the boot menu title, we'll file another
bug for it once the pxe branch is merged to master.

Again, thank you OdyX, you're great. :)




-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#607417: ΑΠ: win32-loader: please offer a "Boot from network with gPXE" option

2011-02-02 Thread Didier 'OdyX' Raboud
Le Wednesday 2 February 2011 14:26:25, vous avez écrit :
> Hi,

Hi Yannis and thanks for your extended review,

> Vaggelis forgot to mention that in order to have the pxe functionality work
> properly you must follow the 1-4 steps:
> 
> 1)The linux, initrd.gz, pxe.lkrn (gpxe 1.0.0) files must be placed in the
> root folder.

I agree that pxe.lkrn has to be placed in the root folder. That's the point of 
the pxe.target to "depend" on pxe.lkrn. I will enhance the code when {g,i}PXE 
enters Debian in order to copy the pxe.lkrn directly (as is done for gl2dr and 
loadlin.exe).

But I disagree with including linux and initrd.gz within the PXE patch, see 
below.

> 2)The file debian/rules must have the PXE=yes option, instead
> of STANDALONE=yes.

… only if you want to build a Debian package out of it. Otherwise, just run 
"PXE=yes make" and you'll get a PXE-enabled win32-loader.exe.

> 3)The Makefile must have:
> 
>   pxe.target: pxe.lkrn linux initrd.gz templates/ternary_choice.ini
> 
>   instead of
>   pxe.target: pxe.lkrn templates/ternary_choice.ini

Same remark as 1), see below.

> 4)The main.nsi should be patched with main.nsi.diff.

(As a side remark, could you please use "diff -Naur" when providing diffs, it's 
way easier to patch back.)

What your main.nsi.diff "patch" does is that it "embeds" a complete debian-
installer linux kernel and initrd.gz (which makes it fail with kfreebsd…). The 
current win32-loader approach is to rely on Debian mirrors (and internet 
connectivity) to get the latest released debian-installer kernels and initrd's. 
Bundling a specific d-i version into a Windows executable defeats the fact that 
the current 0.6.22 win32-loader will be able to download _any_ future "stable" 
Debian-Installer.

But I can acknowledge that bundling specific versions can be useful. Could you 
please report a bug (with wishlist severity) against win32-loader with a title 
like "Please add a d-i bundling possiblity" ?

> Another issue to consider is that when a user installs win32loader in pxe
> mode, description in boot menu isn't very clear ("continue with install
> process"). I think that in pxe mode you should better change this message
> to "Continue with network boot" in file l10n/win32-loader.sh (line 81)

Same here; I plan to enhance the reboot messages shown by the Windows 
bootloader. It'd be nice if you could report a wishlist bug against 
win32-loader 
with a title like "Please enhance the reboot messages."

> Yannis

So, given the messages received so far and my present answer, I am pretty 
confident that the PXE patch does basically what you need; I will proceed in 
merging the pxe branch to "master". Other enhancements will certainly happen, 
but please report separate bugs (it's not a matter of blocking discussion, but 
to keep the "one problem is one bug" motto).

Cheers, 

OdyX
-- 
Didier Raboud, proud Debian Developer.
CH-1020 Renens
o...@debian.org


signature.asc
Description: This is a digitally signed message part.


Bug#607417: ΑΠ: win32-loader: please offer a "Boot from network with gPXE" option

2011-01-28 Thread Didier 'OdyX' Raboud
tags 607417 +pending
thanks

Le Tuesday 11 January 2011 12:40:33 Papakirikou Vaggelis, vous avez écrit :
> Hello,
> 
> I attach the patch for the files we have modified...
> 
> Evangelos Papakirikou (papak...@cti.gr)
> 
> Ioannis Siahos (sia...@cti.gr)

Hi Evangelos, Ioannis, and Alkis, 

(I hope I got the first names correctly…)

Many thanks for your patch; I have now taken some time to inspect it and I
created a branch on the Win32-Loader git repository, that includes most of the
changes you propose:

http://git.debian.org/?p=d-i/win32-loader.git;a=commitdiff;h=61e7dae042308d010ff897c7a7df9e98ac030be7

(on the "pxe" branch)

Here's what I did:
* Add the "PXE" functionality as an option
(aka without PXE=yes make, win32-loader will work as usual)
* Make the functionality gPXE/iPXE independent
(aka you must provide a "pxe.lkrn" file at the sources root)
* No mention of "LTSP", but PXE everywhere
* Add as few new strings as possible
* No change of the Windows Registry key name

I tested that here on a Windows7 machine with a renamed ipxe.lkrn and it just
worked, so many thanks for your patch !

My current plan is to give you some time to test my patch (possibly adding yours
on top) and if it works correctly, I'll merge it on the master branch; hence
releasing a win32-loader source that allows the installation of a PXE loader,
but without enabling it on the binaries (as neither gPXE nor iPXE is in Debian
yet).

So please keep me informed about the correctness of my patch for your purpose !
And by the way, if you could publish your changes in a git repository somewhere,
it'd just be great for more collaboration !

Cheers, 

OdyX

-- 
Didier Raboud, proud Debian Developer.
CH-1020 Renens
o...@debian.org


signature.asc
Description: This is a digitally signed message part.


Bug#607417: ΑΠ: win32-loader: please offer a "Boot from network with gPXE" option

2011-01-11 Thread Papakirikou Vaggelis
Hello,



I attach the patch for the files we have modified...



Evangelos Papakirikou (papak...@cti.gr)

Ioannis Siahos (sia...@cti.gr)









diffs.diff
Description: diffs.diff