Mailing List Archive: compile issues - latest macOS SDK (11.3) / Xcode (12.5)

compile issues - latest macOS SDK (11.3) / Xcode (12.5)

May 30, 2021, 6:02 AM

Post #1 of 4 (492 views)

First - apologies, this is occurring on Xcode 12.5 (latest release)- error
in my original email. I have corrected the subject line and confirmed this
to be a non-issue with Xcode 12.4 and earlier.

> It seems to me that renaming VERSION to something new may be the simplest
> solution.
>

I've put together a fork on github for master and fixes/31 that renames
VERSION to SRC_VERSION and updates version.sh accordingly here:
https://github.com/jhoyt4/mythtv <https://github.com/jhoyt4/mythtv>

I have confirmed that these updates compile successfully on macOS.

If this is an acceptable solution by the community, I'll issue a PR for
both master and fixes/31 and link them to my issue report
<https://github.com/MythTV/mythtv/issues/361>. Also happy to have another
set of eyes look at the update in case I missed something.

If this is not an acceptable solution, no big deal - I will happily delete
the fork. Honestly, I'm just looking for some help / feedback on
alternative solutions if the simple rename is not palatable. After
investigating my compile script, the -l"../.." is NOT passed in by the
script but somewhere else by default in the codebase. This could be
addressed via that route, but I'll need some pointers on where to look.

Re: compile issues - latest macOS SDK (11.3) / Xcode (12.5) [ In reply to ]

mythtv-dev at mythtv

Jun 3, 2021, 7:54 AM

Post #2 of 4 (490 views)

Permalink

On Sun, 2021-05-30 at 09:02 -0400, John Hoyt wrote:
> First - apologies, this is occurring on Xcode 12.5 (latest release)-
> error in my original email. I have corrected the subject line and
> confirmed this to be a non-issue with Xcode 12.4 and earlier.
>
> > It seems to me that renaming VERSION to something new may be the
> > simplest solution.
> >
>
>
> I've put together a fork on github for master and fixes/31 that
> renames VERSION to SRC_VERSION and updates version.sh accordingly
> here: https://github.com/jhoyt4/mythtv
>
> I have confirmed that these updates compile successfully on macOS.
>
> If this is an acceptable solution by the community, I'll issue a PR
> for both master and fixes/31 and link them to my issue report. Also
> happy to have another set of eyes look at the update in case I missed
> something.

Looks good to me.

> If this is not an acceptable solution, no big deal - I will happily
> delete the fork. Honestly, I'm just looking for some help / feedback
> on alternative solutions if the simple rename is not palatable.
> After investigating my compile script, the -l"../.." is NOT passed in
> by the script but somewhere else by default in the codebase. This
> could be addressed via that route, but I'll need some pointers on
> where to look.

Its coming from external/libmythsoundtouch/libmythsoundtouch.pro:

INCLUDEPATH += . ../../

It seems like you should be able to rewrite this as

INCLUDEPATH += $$(PWD) $$(PWD)/../../

to get absolute paths, but its not that simple. (I tried.) QMake is
apparently doing path manipulation (and deduplication?) behind the
scenes, and still insists on including something that starts with ../..
instead of using fully rooted paths. (Weirdly, the ../.. from before
ends up as ../../../mythtv.) Besides, this doesn't solve the problem of
finding "version" in the system include path before finding "VERSION"
in the user include path.

If no-one else has any objection, I can commit your changes.

David

_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org

Re: compile issues - latest macOS SDK (11.3) / Xcode (12.5) [ In reply to ]

john.hoyt at gmail

Jun 5, 2021, 4:27 AM

Post #3 of 4 (488 views)

Permalink

On Thu, Jun 3, 2021 at 10:55 AM David Hampton via mythtv-dev <
mythtv-dev@mythtv.org> wrote:

> On Sun, 2021-05-30 at 09:02 -0400, John Hoyt wrote:
> > First - apologies, this is occurring on Xcode 12.5 (latest release)-
> > error in my original email. I have corrected the subject line and
> > confirmed this to be a non-issue with Xcode 12.4 and earlier.
> >
> > > It seems to me that renaming VERSION to something new may be the
> > > simplest solution.
> > >
> >
> >
> > I've put together a fork on github for master and fixes/31 that
> > renames VERSION to SRC_VERSION and updates version.sh accordingly
> > here: https://github.com/jhoyt4/mythtv
> >
> > I have confirmed that these updates compile successfully on macOS.
> >
> > If this is an acceptable solution by the community, I'll issue a PR
> > for both master and fixes/31 and link them to my issue report. Also
> > happy to have another set of eyes look at the update in case I missed
> > something.
>
> Looks good to me.
>
> > If this is not an acceptable solution, no big deal - I will happily
> > delete the fork. Honestly, I'm just looking for some help / feedback
> > on alternative solutions if the simple rename is not palatable.
> > After investigating my compile script, the -l"../.." is NOT passed in
> > by the script but somewhere else by default in the codebase. This
> > could be addressed via that route, but I'll need some pointers on
> > where to look.
>
> Its coming from external/libmythsoundtouch/libmythsoundtouch.pro:
>
> INCLUDEPATH += . ../../
>
> It seems like you should be able to rewrite this as
>
> INCLUDEPATH += $$(PWD) $$(PWD)/../../
>
> to get absolute paths, but its not that simple. (I tried.) QMake is
> apparently doing path manipulation (and deduplication?) behind the
> scenes, and still insists on including something that starts with ../..
> instead of using fully rooted paths. (Weirdly, the ../.. from before
> ends up as ../../../mythtv.) Besides, this doesn't solve the problem of
> finding "version" in the system include path before finding "VERSION"
> in the user include path.
>
> If no-one else has any objection, I can commit your changes.
>

David - thank you for looking in on this. Apple has stopped responding to
my issue ticket, but I'm going to keep at it to hopefully get this
corrected path ordering correct.

Re: compile issues - latest macOS SDK (11.3) / Xcode (12.5) [ In reply to ]

gary.buhrmaster at gmail

Jun 6, 2021, 3:21 PM

Post #4 of 4 (488 views)

Permalink

On Thu, Jun 3, 2021 at 2:55 PM David Hampton via mythtv-dev
<mythtv-dev@mythtv.org> wrote:

> Its coming from external/libmythsoundtouch/libmythsoundtouch.pro:
>
> INCLUDEPATH += . ../../
>
> It seems like you should be able to rewrite this as
>
> INCLUDEPATH += $$(PWD) $$(PWD)/../../
>
> to get absolute paths, but its not that simple. (I tried.)

With qmake, it never seems to be easy
to find the magic incantation that makes
things work the way one wants.

I sort of wish that the automatic qmake
to cmake converters would actually work
well across a project as large as MythTV
(Note that I also want the configure script
to disappear). Not that I am a huge cmake
fan, but at least I imagine I can more
easily understand the magic.
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org