<div dir="ltr"><div>Hi again,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Sa., 30. Mai 2020 um 19:28 Uhr schrieb Mario Mintel <<a href="mailto:mariomintel@gmail.com">mariomintel@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi everyone,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Fr., 29. Mai 2020 um 23:40 Uhr schrieb Ralf Ramsauer <<a href="mailto:ralf.ramsauer@oth-regensburg.de" target="_blank">ralf.ramsauer@oth-regensburg.de</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Yann,<br>
<br>
On 5/29/20 10:48 PM, Yann E. MORIN wrote:<br>
> Mario, All,<br>
> <br>
> On 2020-05-28 16:43 +0200, Mario Mintel spake thusly:<br>
>> In addition to official releases of Jailhouse, allow to specify a custom<br>
>> Git URI + branches. This adds more flexibility for custom<br>
>> configurations.<br>
> <br>
> The overwhelming majority of packages do not allow selecting an<br>
> alternate location. Why would jailhouse be different?<br>
<br>
Jailhouse requires system-specific configurations. Those configurations<br>
are compiled from C source files to binaries during the build process.<br>
While upstream Jailhouse comes with a lot of samples for supported<br>
systems, you will need a lot of fine tuning to for a specific use case.<br>
<br>
> <br>
> Are you trying to cover for development? In that case, the usual way is<br>
<br>
In our case -- Yes.<br>
<br>
> to use the override srcdir mechanism. See BR2_PACKAGE_OVERRIDE_FILE and<br>
> provide such a file with definitions like;<br>
> <br>
>     JAILHOUSE_OVERRIDE_SRCDIR = /path/to/your/local/development/tree/jailhouse<br>
<br>
Oh, that could maybe work. Mario, could you please check that? In case<br>
this works, we can simply add those definitions to our br2-external tree.<br>
<br></blockquote><div><br></div><div><br>Yeah sure I can try that. Although I won't be able to until Monday as I have no<br>access to my working computer as of right now.<br><br>I will report back as soon as I have results.</div></div></div></blockquote><div><br></div><div>It does work as proposed by Yann. I wasn't aware of that option. I guess that makes this patch redundant.<br></div><div><br></div><div>Greetings,</div><div>Mario<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>Greetings,</div><div>Mario<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Thanks<br>
  Ralf<br>
<br>
> <br>
> Regards,<br>
> Yann E. MORIN.<br>
> <br>
>> Signed-off-by: Mario Mintel <<a href="mailto:mariomintel@gmail.com" target="_blank">mariomintel@gmail.com</a>><br>
>> ---<br>
>>  package/jailhouse/Config.in    | 36 ++++++++++++++++++++++++++++++++--<br>
>>  package/jailhouse/<a href="http://jailhouse.mk" rel="noreferrer" target="_blank">jailhouse.mk</a> | 17 ++++++++++++++--<br>
>>  2 files changed, 49 insertions(+), 4 deletions(-)<br>
>><br>
>> diff --git a/package/jailhouse/Config.in b/package/jailhouse/Config.in<br>
>> index 596b4951db..47523747f9 100644<br>
>> --- a/package/jailhouse/Config.in<br>
>> +++ b/package/jailhouse/Config.in<br>
>> @@ -3,18 +3,50 @@ config BR2_PACKAGE_JAILHOUSE<br>
>>      depends on BR2_aarch64 || BR2_x86_64<br>
>>      depends on BR2_LINUX_KERNEL<br>
>>      help<br>
>> -      The Jailhouse partitioning Hypervisor based on Linux.<br>
>> +      The Jailhouse Linux-based partitioning hypervisor.<br>
>>  <br>
>>        <a href="https://github.com/siemens/jailhouse" rel="noreferrer" target="_blank">https://github.com/siemens/jailhouse</a><br>
>>  <br>
>>  if BR2_PACKAGE_JAILHOUSE<br>
>>  <br>
>> +choice<br>
>> +    prompt "Jailhouse Version"<br>
>> +<br>
>> +config BR2_PACKAGE_JAILHOUSE_LATEST_VERSION<br>
>> +    bool "Version 0.12"<br>
>> +<br>
>> +config BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT<br>
>> +    bool "Custom Git repository"<br>
>> +    help<br>
>> +      This option allows Buildroot to get the Jailhouse source code<br>
>> +      from a custom Git repository.<br>
>> +<br>
>> +endchoice<br>
>> +<br>
>> +if BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT<br>
>> +<br>
>> +config BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT_URI<br>
>> +    string "URI of custom repository"<br>
>> +    default "<a href="https://github.com/siemens/jailhouse.git" rel="noreferrer" target="_blank">https://github.com/siemens/jailhouse.git</a>"<br>
>> +<br>
>> +config BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT_BRANCH<br>
>> +    string "Name of Git branch"<br>
>> +    default "master"<br>
>> +<br>
>> +endif<br>
>> +<br>
>> +config BR2_PACKAGE_JAILHOUSE_VERSION<br>
>> +    string<br>
>> +    default "0.12" if BR2_PACKAGE_JAILHOUSE_LATEST_VERSION<br>
>> +    default BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT_BRANCH \<br>
>> +            if BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT<br>
>> +<br>
>>  config BR2_PACKAGE_JAILHOUSE_HELPER_SCRIPTS<br>
>>      bool "helper scripts"<br>
>>      depends on BR2_PACKAGE_PYTHON<br>
>>      select BR2_PACKAGE_PYTHON_MAKO # runtime<br>
>>      help<br>
>> -      Python-based helpers for the Jailhouse Hypervisor.<br>
>> +      Python-based helpers for the Jailhouse hypervisor.<br>
>>  <br>
>>        <a href="https://github.com/siemens/jailhouse" rel="noreferrer" target="_blank">https://github.com/siemens/jailhouse</a><br>
>>  <br>
>> diff --git a/package/jailhouse/<a href="http://jailhouse.mk" rel="noreferrer" target="_blank">jailhouse.mk</a> b/package/jailhouse/<a href="http://jailhouse.mk" rel="noreferrer" target="_blank">jailhouse.mk</a><br>
>> index 6356c5a7aa..d134b3d1b4 100644<br>
>> --- a/package/jailhouse/<a href="http://jailhouse.mk" rel="noreferrer" target="_blank">jailhouse.mk</a><br>
>> +++ b/package/jailhouse/<a href="http://jailhouse.mk" rel="noreferrer" target="_blank">jailhouse.mk</a><br>
>> @@ -4,10 +4,23 @@<br>
>>  #<br>
>>  ################################################################################<br>
>>  <br>
>> -JAILHOUSE_VERSION = 0.12<br>
>> -JAILHOUSE_SITE = $(call github,siemens,jailhouse,v$(JAILHOUSE_VERSION))<br>
>> +JAILHOUSE_VERSION = $(call qstrip,$(BR2_PACKAGE_JAILHOUSE_VERSION))<br>
>>  JAILHOUSE_LICENSE = GPL-2.0<br>
>> +ifeq ($(BR2_PACKAGE_JAILHOUSE_LATEST_VERSION),y)<br>
>>  JAILHOUSE_LICENSE_FILES = COPYING<br>
>> +endif<br>
>> +<br>
>> +ifeq ($(BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT),y)<br>
>> +JAILHOUSE_SITE = $(call qstrip,$(BR2_PACKAGE_JAILHOUSE_CUSTOM_GIT_URI))<br>
>> +JAILHOUSE_SITE_METHOD = git<br>
>> +else<br>
>> +JAILHOUSE_SITE = $(call github,siemens,jailhouse,v$(JAILHOUSE_VERSION))<br>
>> +endif<br>
>> +<br>
>> +ifeq ($(BR2_PACKAGE_JAILHOUSE)$(BR2_PACKAGE_JAILHOUSE_CUSTIM_GIT),y)<br>
>> +BR_NO_CHECK_HASH_FOR += $(JAILHOUSE_SOURCE)<br>
>> +endif<br>
>> +<br>
>>  JAILHOUSE_DEPENDENCIES = \<br>
>>      linux<br>
>>  <br>
>> -- <br>
>> 2.26.1<br>
>><br>
>> _______________________________________________<br>
>> buildroot mailing list<br>
>> <a href="mailto:buildroot@busybox.net" target="_blank">buildroot@busybox.net</a><br>
>> <a href="http://lists.busybox.net/mailman/listinfo/buildroot" rel="noreferrer" target="_blank">http://lists.busybox.net/mailman/listinfo/buildroot</a><br>
>  <br></blockquote></div></div>
</blockquote></div></div>