All of lore.kernel.org
 help / color / mirror / Atom feed
* Is the a reason for adding the + to SRCPV?
@ 2018-04-10  8:29 Måns Zigher
  2018-04-10 12:44 ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Måns Zigher @ 2018-04-10  8:29 UTC (permalink / raw)
  To: Yocto discussion list

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

Hi,

I am having some problems with one of my recipes when using the SRCPV the
get_srcrev is returning "AUTOINC+" + rev. This will cause problem in my
compile because I am using dbus codegen and it looks like the
auto-generated code cannot handle the + sign in the paths. Is there a
reason for this + sign. The generated code that causes my build to fail
looks like this

#ifndef
___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__

Currently the only way for me to get the build to work is not use the
SRCPV. Could it actually be a better way then using the + sign or is there
a logical requirement for it?

BR
Mans Zigher

[-- Attachment #2: Type: text/html, Size: 2933 bytes --]

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

* Re: Is the a reason for adding the + to SRCPV?
  2018-04-10  8:29 Is the a reason for adding the + to SRCPV? Måns Zigher
@ 2018-04-10 12:44 ` Burton, Ross
  2018-04-11  9:12   ` Måns Zigher
  0 siblings, 1 reply; 6+ messages in thread
From: Burton, Ross @ 2018-04-10 12:44 UTC (permalink / raw)
  To: Måns Zigher; +Cc: Yocto discussion list

What tool is generating the code?  It probably shouldn't be using the
full path of the header...

Ross

On 10 April 2018 at 09:29, Måns Zigher <mans.zigher@gmail.com> wrote:
> Hi,
>
> I am having some problems with one of my recipes when using the SRCPV the
> get_srcrev is returning "AUTOINC+" + rev. This will cause problem in my
> compile because I am using dbus codegen and it looks like the auto-generated
> code cannot handle the + sign in the paths. Is there a reason for this +
> sign. The generated code that causes my build to fail looks like this
>
> #ifndef
> ___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__
>
> Currently the only way for me to get the build to work is not use the SRCPV.
> Could it actually be a better way then using the + sign or is there a
> logical requirement for it?
>
> BR
> Mans Zigher
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
>


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

* Re: Is the a reason for adding the + to SRCPV?
  2018-04-10 12:44 ` Burton, Ross
@ 2018-04-11  9:12   ` Måns Zigher
  2018-04-11 13:49     ` Burton, Ross
  0 siblings, 1 reply; 6+ messages in thread
From: Måns Zigher @ 2018-04-11  9:12 UTC (permalink / raw)
  To: Burton, Ross; +Cc: Yocto discussion list

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

I believe dbus codegen is the tools generating the code.

Br
Mans Zigher

On Tue, Apr 10, 2018, 14:44 Burton, Ross <ross.burton@intel.com> wrote:

> What tool is generating the code?  It probably shouldn't be using the
> full path of the header...
>
> Ross
>
> On 10 April 2018 at 09:29, Måns Zigher <mans.zigher@gmail.com> wrote:
> > Hi,
> >
> > I am having some problems with one of my recipes when using the SRCPV the
> > get_srcrev is returning "AUTOINC+" + rev. This will cause problem in my
> > compile because I am using dbus codegen and it looks like the
> auto-generated
> > code cannot handle the + sign in the paths. Is there a reason for this +
> > sign. The generated code that causes my build to fail looks like this
> >
> > #ifndef
> >
> ___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__
> >
> > Currently the only way for me to get the build to work is not use the
> SRCPV.
> > Could it actually be a better way then using the + sign or is there a
> > logical requirement for it?
> >
> > BR
> > Mans Zigher
> >
> > --
> > _______________________________________________
> > yocto mailing list
> > yocto@yoctoproject.org
> > https://lists.yoctoproject.org/listinfo/yocto
> >
>

[-- Attachment #2: Type: text/html, Size: 2143 bytes --]

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

* Re: Is the a reason for adding the + to SRCPV?
  2018-04-11  9:12   ` Måns Zigher
@ 2018-04-11 13:49     ` Burton, Ross
  2018-04-11 15:59       ` Måns Zigher
  0 siblings, 1 reply; 6+ messages in thread
From: Burton, Ross @ 2018-04-11 13:49 UTC (permalink / raw)
  To: Måns Zigher; +Cc: Yocto discussion list

There is no "dbus codegen".  dbus-glib has a code generator but that
doesn't write include guards like that, so it must be something else.
gdbus-codegen appears to be doing the right thing to unless it breaks
if you pass it an absolute path?

Ross

On 11 April 2018 at 10:12, Måns Zigher <mans.zigher@gmail.com> wrote:
> I believe dbus codegen is the tools generating the code.
>
> Br
> Mans Zigher
>
> On Tue, Apr 10, 2018, 14:44 Burton, Ross <ross.burton@intel.com> wrote:
>>
>> What tool is generating the code?  It probably shouldn't be using the
>> full path of the header...
>>
>> Ross
>>
>> On 10 April 2018 at 09:29, Måns Zigher <mans.zigher@gmail.com> wrote:
>> > Hi,
>> >
>> > I am having some problems with one of my recipes when using the SRCPV
>> > the
>> > get_srcrev is returning "AUTOINC+" + rev. This will cause problem in my
>> > compile because I am using dbus codegen and it looks like the
>> > auto-generated
>> > code cannot handle the + sign in the paths. Is there a reason for this +
>> > sign. The generated code that causes my build to fail looks like this
>> >
>> > #ifndef
>> >
>> > ___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__
>> >
>> > Currently the only way for me to get the build to work is not use the
>> > SRCPV.
>> > Could it actually be a better way then using the + sign or is there a
>> > logical requirement for it?
>> >
>> > BR
>> > Mans Zigher
>> >
>> > --
>> > _______________________________________________
>> > yocto mailing list
>> > yocto@yoctoproject.org
>> > https://lists.yoctoproject.org/listinfo/yocto
>> >


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

* Re: Is the a reason for adding the + to SRCPV?
  2018-04-11 13:49     ` Burton, Ross
@ 2018-04-11 15:59       ` Måns Zigher
  2018-05-04 19:19         ` Martin Jansa
  0 siblings, 1 reply; 6+ messages in thread
From: Måns Zigher @ 2018-04-11 15:59 UTC (permalink / raw)
  To: Burton, Ross; +Cc: Yocto discussion list

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

Hi,

Thanks for your help. It is a cmake project in which they have specified a
custom command calling gdbus-codegen. It definitely looks like they are
passing the absolute path to it thanks for the help I think I can solve
that.

Br
Mans Zigher

On Apr 11, 2018 15:50, "Burton, Ross" <ross.burton@intel.com> wrote:

There is no "dbus codegen".  dbus-glib has a code generator but that
doesn't write include guards like that, so it must be something else.
gdbus-codegen appears to be doing the right thing to unless it breaks
if you pass it an absolute path?


Ross


On 11 April 2018 at 10:12, Måns Zigher <mans.zigher@gmail.com> wrote:
> I believe dbus codegen is the tools generating the code.
>
> Br
> Mans Zigher
>
> On Tue, Apr 10, 2018, 14:44 Burton, Ross <ross.burton@intel.com> wrote:
>>
>> What tool is generating the code?  It probably shouldn't be using the
>> full path of the header...
>>
>> Ross
>>
>> On 10 April 2018 at 09:29, Måns Zigher <mans.zigher@gmail.com> wrote:
>> > Hi,
>> >
>> > I am having some problems with one of my recipes when using the SRCPV
>> > the
>> > get_srcrev is returning "AUTOINC+" + rev. This will cause problem in my
>> > compile because I am using dbus codegen and it looks like the
>> > auto-generated
>> > code cannot handle the + sign in the paths. Is there a reason for this
+
>> > sign. The generated code that causes my build to fail looks like this
>> >
>> > #ifndef
>> >
>> >
___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__
>> >
>> > Currently the only way for me to get the build to work is not use the
>> > SRCPV.
>> > Could it actually be a better way then using the + sign or is there a
>> > logical requirement for it?
>> >
>> > BR
>> > Mans Zigher
>> >
>> > --
>> > _______________________________________________
>> > yocto mailing list
>> > yocto@yoctoproject.org
>> > https://lists.yoctoproject.org/listinfo/yocto
>> >

[-- Attachment #2: Type: text/html, Size: 3891 bytes --]

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

* Re: Is the a reason for adding the + to SRCPV?
  2018-04-11 15:59       ` Måns Zigher
@ 2018-05-04 19:19         ` Martin Jansa
  0 siblings, 0 replies; 6+ messages in thread
From: Martin Jansa @ 2018-05-04 19:19 UTC (permalink / raw)
  To: Måns Zigher; +Cc: Yocto discussion list

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

This might help you:
https://github.com/webosose/meta-webosose/blob/master/meta-webos/recipes-core/glib-2.0/glib-2.0/0002-gdbus-codegen-also-replace-character-with-underscore.patch

On Wed, Apr 11, 2018 at 5:59 PM, Måns Zigher <mans.zigher@gmail.com> wrote:

> Hi,
>
> Thanks for your help. It is a cmake project in which they have specified a
> custom command calling gdbus-codegen. It definitely looks like they are
> passing the absolute path to it thanks for the help I think I can solve
> that.
>
> Br
> Mans Zigher
>
> On Apr 11, 2018 15:50, "Burton, Ross" <ross.burton@intel.com> wrote:
>
> There is no "dbus codegen".  dbus-glib has a code generator but that
> doesn't write include guards like that, so it must be something else.
> gdbus-codegen appears to be doing the right thing to unless it breaks
> if you pass it an absolute path?
>
>
> Ross
>
>
> On 11 April 2018 at 10:12, Måns Zigher <mans.zigher@gmail.com> wrote:
> > I believe dbus codegen is the tools generating the code.
> >
> > Br
> > Mans Zigher
> >
> > On Tue, Apr 10, 2018, 14:44 Burton, Ross <ross.burton@intel.com> wrote:
> >>
> >> What tool is generating the code?  It probably shouldn't be using the
> >> full path of the header...
> >>
> >> Ross
> >>
> >> On 10 April 2018 at 09:29, Måns Zigher <mans.zigher@gmail.com> wrote:
> >> > Hi,
> >> >
> >> > I am having some problems with one of my recipes when using the SRCPV
> >> > the
> >> > get_srcrev is returning "AUTOINC+" + rev. This will cause problem in
> my
> >> > compile because I am using dbus codegen and it looks like the
> >> > auto-generated
> >> > code cannot handle the + sign in the paths. Is there a reason for
> this +
> >> > sign. The generated code that causes my build to fail looks like this
> >> >
> >> > #ifndef
> >> >
> >> > ___HOME_EXTZIG_WORKSPACE_MOZART_MOZART_WORKSPACE_
> BUILDS_MOZART_RASPBERRYPI_BUILD_TMP_WORK_CORTEXA7HF_
> NEON_VFPV4_MOZART_LINUX_GNUEABI_MOZART_DAEMON_1_0_
> GITAUTOINC+8FD4C803AE_R3_GIT_SRC_LIBS_MOZART_BLE_SETUP_
> CODEGEN_ORG_BLUEZ_LE_ADVERTISEMENT_INTERFACE_H__
> >> >
> >> > Currently the only way for me to get the build to work is not use the
> >> > SRCPV.
> >> > Could it actually be a better way then using the + sign or is there a
> >> > logical requirement for it?
> >> >
> >> > BR
> >> > Mans Zigher
> >> >
> >> > --
> >> > _______________________________________________
> >> > yocto mailing list
> >> > yocto@yoctoproject.org
> >> > https://lists.yoctoproject.org/listinfo/yocto
> >> >
>
>
>
> --
> _______________________________________________
> yocto mailing list
> yocto@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
>
>

[-- Attachment #2: Type: text/html, Size: 5106 bytes --]

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

end of thread, other threads:[~2018-05-04 19:19 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-10  8:29 Is the a reason for adding the + to SRCPV? Måns Zigher
2018-04-10 12:44 ` Burton, Ross
2018-04-11  9:12   ` Måns Zigher
2018-04-11 13:49     ` Burton, Ross
2018-04-11 15:59       ` Måns Zigher
2018-05-04 19:19         ` Martin Jansa

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.