Re: memleax spec file and review

2021-04-06 Thread Germano Massullo
Done!
Review Request: memleax - Debugs memory leak of a running process -
https://bugzilla.redhat.com/1946499
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: memleax spec file and review

2021-04-06 Thread Germano Massullo
Il 04/04/21 23:55, Ian McInerney ha scritto:
>
> On Sun, 4 Apr 2021, 21:53 Germano Massullo,
> mailto:germano.massu...@gmail.com>> wrote:
>
> Good day, I am creating a spec file [0] for memleax memory leaks
> analyzer [1], but during build [2] I am getting error "invalid option:
> --build=x86_64-redhat-linux-gnu.". Where can be the problem?
> Thank you
>
> [0]: https://pagure.io/memleax/blob/master/f/memleax.spec
> 
> [1]: https://github.com/WuBingzheng/memleax
> 
> [2]:
> https://copr.fedorainfracloud.org/coprs/germano/memleax/build/2114611/
> 
>
>
> The GitHub repo you linked at [1] doesn't seem to have any
> autotools-based build systems, so you shouldn't use the %configure
> macro. It looks like it uses CMake, so you should instead use the
> %cmake_* macros to configure/build/install.

The tar.gz only contains configure.ac, that is not included in master
branch. Also, CMakeLists.txt included in master branch, is not included
in tar.gz file.
I now added the latter as Source1 and I am working on let RPM use such
file, I just have to find out how to properly place the cp command in
the spec file

[0]: https://pagure.io/memleax/blob/master/f/memleax.spec

___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: memleax spec file and review

2021-04-04 Thread Ian McInerney
On Sun, 4 Apr 2021, 21:53 Germano Massullo, 
wrote:

> Good day, I am creating a spec file [0] for memleax memory leaks
> analyzer [1], but during build [2] I am getting error "invalid option:
> --build=x86_64-redhat-linux-gnu.". Where can be the problem?
> Thank you
>
> [0]: https://pagure.io/memleax/blob/master/f/memleax.spec
> [1]: https://github.com/WuBingzheng/memleax
> [2]:
> https://copr.fedorainfracloud.org/coprs/germano/memleax/build/2114611/


The GitHub repo you linked at [1] doesn't seem to have any autotools-based
build systems, so you shouldn't use the %configure macro. It looks like it
uses CMake, so you should instead use the %cmake_* macros to
configure/build/install.

-Ian
___
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure