Re: [Xen-devel] [PATCH] build: remove tboot make targets

[Christopher Clark <christopher.w.clark@xxxxxxxxx>]

On Mon, Aug 20, 2018 at 1:38 AM, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
> On Sat, Aug 18, 2018 at 09:22:05PM -0500, Doug Goldstein wrote:
>> The tboot targets are woefully out of date. These should really be
>> retired because setting up tboot is more complex than the build process
>> for it.
>>
>> Signed-off-by: Doug Goldstein <cardoe@xxxxxxxxxx>
>
> Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>

Code reviewed and standard (non-tboot related) build tested with the
patch applied: looks good.

No references to any of the following remain in the tree afterwards:
* TBOOT_TARFILE
* TBOOT_BASE_URL
* build-tboot, install-tboot, dist-tboot, clean-tboot, distclean-tboot
* download_tboot

The README section about tboot, just above the chunk removed, contains
a stale URL for the tboot hg repository, which has apparently now
moved to:
http://hg.code.sf.net/p/tboot/code

I don't use uninstall (ever) but since Doug's patch touches it, I just
tested it with this applied. Uninstall failed, but it's not a fault of
this patch:

| make[6]: Entering directory './git/tools/tests/x86_emulator'
| make[6]: *** No rule to make target 'uninstall'.  Stop.
| make[6]: Leaving directory './git/tools/tests/x86_emulator'
|
| make[6]: *** No rule to make target 'uninstall'.  Stop.
| make[6]: Leaving directory '/./git/tools/tests/x86_emulator'
| ./git/tools/tests/../../tools/Rules.mk:249: recipe for target
'subdir-uninstall-x86_emulator' failed
| make[5]: *** [subdir-uninstall-x86_emulator] Error 2
| make[5]: Leaving directory './git/tools/tests'
|
| ./git/tools/tests/../../tools/Rules.mk:249: recipe for target
'subdir-uninstall-x86_emulator' failed

so I've just posted a patch that fixes this and another defect in the
Linux hotplug uninstall logic to make it actually remove what it
installed.

Reviewed-by: Christopher Clark <christopher.clark6@xxxxxxxxxxxxxx>

Christopher

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel