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 (firstname.lastname@example.org) > > Ioannis Siahos (email@example.com) 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 firstname.lastname@example.org
Description: This is a digitally signed message part.