custom.pro does not produces effects

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

custom.pro does not produces effects

Nicu Tofan
custom.pro is included in librecad.pro by this line:

exists( custom.pro ):include( custom.pro )

I suspect that the author expected that the things defined in custom.pro will be visible to the projects in sub-directories. However, this is not the case (I don't know if this is a bug in qmake or it is so by design). To make it useful above line should be copied to all .pro files.

I use this file kind of to indicate library and include paths on Windows and is quite useful to me.

I'm posting here instead of a pull request because the purpose of this file may be other than the one that I assume. Is it ok to add the line:

# custom definitions from the user
exists( custom.pro ):include( custom.pro )

in common.pro? This will include custom.pro in all projects that produce output:
- jwwlib
- libdxfrw
- src
- all plugins
- ttf2ltf
Reply | Threaded
Open this post in threaded view
|

Re: custom.pro does not produces effects

R. van Twisk
Administrator
CONTENTS DELETED
The author has deleted this message.
Reply | Threaded
Open this post in threaded view
|

Re: custom.pro does not produces effects

Nicu Tofan
Hey, thanks for the feedback.

I have no intention to suggest automake or other build system, I'm just fine with qmake.

Being the original author, may I ask what is the purpose of custom.pro? Is it ok to use it the way I presented it? To be a place where local customizations are stored? Would it be OK to add it to .gitignore?

I'm only asking because this may not be the intended purpose for that file, and, if this is not the purpose, then I would suggest creating one such file that does not throw git warnings about new files not being added. I see it as a very clean way of customizing Qt driven projects.
Reply | Threaded
Open this post in threaded view
|

Re: custom.pro does not produces effects

Nicu Tofan
I've forwarded a pull request on this matter here.