opus - Apr 2019 - CMake support for Opus proposal (in addition to Autotools)

Hi,


Here is a proposal for adding CMake in addition of Autotools.


See the attached patch for changes or the pull request here:
https://github.com/xiph/opus/pull/100


The CMake scripts are parsing the filelist from Autotools, so only one file list
needs to be maintained.


With this change the checked in Visual Studio solution can be removed and no
longer need to be maintained and updated when Visual Studio is updated.


For related discussion about CMake support see
https://github.com/xiph/opus/pull/37


For any concerns, comments or questions regarding the change please comment in
the pull request, reply to this thread or send me an e-mail.


Related Issue:

https://github.com/xiph/opus/issues/85


//Marcus

-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.xiph.org/pipermail/opus/attachments/20190402/c89b88c8/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Adding-CMake-support-for-Windows-Mac-Linux-and-Andro.patch
Type: application/octet-stream
Size: 28868 bytes
Desc: 0001-Adding-CMake-support-for-Windows-Mac-Linux-and-Andro.patch
URL:
<http://lists.xiph.org/pipermail/opus/attachments/20190402/c89b88c8/attachment-0001.obj>

Hi Marcus,

There seems to be a conflict between the autotools-created config.h.in
and the same file added by your patch. Any way you can use use a
different name?

Cheers,

	Jean-Marc

On 04/02/2019 01:39 AM, Marcus Asteborg wrote:
> Hi,
> 
> 
> Here is a proposal for adding CMake in addition of Autotools.
> 
> 
> See the attached patch for changes or the pull request here:
> https://github.com/xiph/opus/pull/100
> <https://github.com/xiph/opus/pull/100>
> 
> 
> The CMake scripts are parsing the filelist from Autotools, so only one
> file list needs to be maintained.
> 
> 
> With this change the checked in Visual Studio solution can be removed
> and no longer need to be maintained and updated when Visual Studio is
> updated.
> 
> 
> For related discussion about CMake support see
> https://github.com/xiph/opus/pull/37
<https://github.com/xiph/opus/pull/37>
> 
> 
> For any concerns, comments or questions regarding the change please
> comment in the pull request, reply to this thread or send me an e-mail.
> 
> 
> Related Issue:
> 
> https://github.com/xiph/opus/issues/85
> <https://github.com/xiph/opus/issues/85>
> 
> 
> //Marcus
> 
> 
> 
> 
> _______________________________________________
> opus mailing list
> opus at xiph.org
> http://lists.xiph.org/mailman/listinfo/opus
>

Hi,

Thanks for the quick feedback I have changed the name to avoid conflict.

Attached is the updated patch.

//Marcus
________________________________
From: Jean-Marc Valin <jmvalin at jmvalin.ca>
Sent: Tuesday, April 2, 2019 00:07
To: Marcus Asteborg; opus at xiph.org
Subject: Re: [opus] CMake support for Opus proposal (in addition to Autotools)

Hi Marcus,

There seems to be a conflict between the autotools-created config.h.in
and the same file added by your patch. Any way you can use use a
different name?

Cheers,

        Jean-Marc

On 04/02/2019 01:39 AM, Marcus Asteborg wrote:
> Hi,
>
>
> Here is a proposal for adding CMake in addition of Autotools.
>
>
> See the attached patch for changes or the pull request here:
>
https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fpull%2F100&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328827874&amp;sdata=ulSL9eGKj8cVt6VPufmkod061Ii56r%2FAsFXvOjyWu44%3D&amp;reserved=0
>
<https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fpull%2F100&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328827874&amp;sdata=ulSL9eGKj8cVt6VPufmkod061Ii56r%2FAsFXvOjyWu44%3D&amp;reserved=0>
>
>
> The CMake scripts are parsing the filelist from Autotools, so only one
> file list needs to be maintained.
>
>
> With this change the checked in Visual Studio solution can be removed
> and no longer need to be maintained and updated when Visual Studio is
> updated.
>
>
> For related discussion about CMake support see
>
https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fpull%2F37&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328827874&amp;sdata=t1IJUHKLyyDpGWzyi8A6dozfmFDZalgHs8Ds%2Bnlf9Qc%3D&amp;reserved=0
<https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fpull%2F37&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328827874&amp;sdata=t1IJUHKLyyDpGWzyi8A6dozfmFDZalgHs8Ds%2Bnlf9Qc%3D&amp;reserved=0>
>
>
> For any concerns, comments or questions regarding the change please
> comment in the pull request, reply to this thread or send me an e-mail.
>
>
> Related Issue:
>
>
https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fissues%2F85&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328827874&amp;sdata=MUh7c1xua91%2Bvn8M33fbsTKS8%2FZeyZV02BorkANy6h4%3D&amp;reserved=0
>
<https://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxiph%2Fopus%2Fissues%2F85&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328837885&amp;sdata=rQA6M3kf87Zd%2Bc3IBVExdUWj8zKYMIzfTAePmrXDnwo%3D&amp;reserved=0>
>
>
> //Marcus
>
>
>
>
> _______________________________________________
> opus mailing list
> opus at xiph.org
>
https://nam01.safelinks.protection.outlook.com/?url=http%3A%2F%2Flists.xiph.org%2Fmailman%2Flistinfo%2Fopus&amp;data=02%7C01%7C%7C47f897fd8d6e42c8336c08d6b739d457%7C84df9e7fe9f640afb435aaaaaaaaaaaa%7C1%7C0%7C636897856328837885&amp;sdata=xTw1jcE0ABlREHtDX5ycQXcrYnzQijLEWJ2LZk6JPEk%3D&amp;reserved=0
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.xiph.org/pipermail/opus/attachments/20190402/97d29304/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Adding-CMake-support-for-Windows-Mac-Linux-and-Andro.patch
Type: application/octet-stream
Size: 28898 bytes
Desc: 0002-Adding-CMake-support-for-Windows-Mac-Linux-and-Andro.patch
URL:
<http://lists.xiph.org/pipermail/opus/attachments/20190402/97d29304/attachment-0001.obj>