Opened 4 years ago
Closed 4 years ago
#13542 closed Patch - Feature (fixed)
Feature patch - qmake .pro file modernization for various non-app directories
| Reported by: | Gary Buhrmaster | Owned by: | David Hampton |
|---|---|---|---|
| Priority: | minor | Milestone: | 31.0 |
| Component: | MythTV - General | Version: | Master Head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
Feature patch - qmake .pro file modernization for various non-app directories
Change qmake TEMPLATE to aux (from app) for directories that do not build anything (and remove app template artifacts).
The aux template was added in QT5. Per the docs the aux template "Creates a Makefile for not building anything. Use this if no compiler needs to be invoked to create the target" which is exactly what we want for these directories.
The patch was mostly mechanical (update the .pro file, remove the dummy.c source file).
*SPECIAL NOTE* These updates should arguably also update the .gitignore files in the respective directories to no longer exclude the target binary (created from dummy.c), but to remove that target binary would require every developer to run a make clean/distclean before pulling the commit to avoid the possibility of accidentally adding that (now leftover) binary into the project at some future commit. As the cost of the .gitignore entry is small, I made the choice to not update .gitignore in the directories. If the committer is confident that there will be no confusion, removing (or emptying the contents of) the .gitignore from the impacted directories can be performed. Caveat Emptor.
Patch attached.
Attachments (1)
Change History (3)
Changed 4 years ago by
| Attachment: | mythtv-template-modernization.patch added |
|---|
comment:1 Changed 4 years ago by
| Milestone: | needs_triage → 31.0 |
|---|---|
| Owner: | set to David Hampton |
| Status: | new → assigned |
comment:2 Changed 4 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
qmake .pro file modernization