Modify

Ticket #1295 (closed patch: wontfix)

You must read the TicketHowTo before creating a new ticket or commenting on an existing ticket.

Opened 6 years ago

Last modified 5 years ago

Patch to allow seperate build directory

Reported by: Dibblah Owned by: ijr
Priority: trivial Milestone: 0.21
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

This patch allows the following to work:

mkdir ~/mythbuild
cd ~/mythbuild
/usr/src/mythtv/configure
make
make install
  • the source directory is completely seperated from the build directory. The source directory is not modified during the build.

It's fairly hacky, but consistently so:

qmake, when it include ( )s a file, changes into the directory of that file first. This means that it's not possible to easily centralise these changes.

Headers do not seem to get pulled in from the VPATH if they're located away from the current build directory - Hence the modifications to the INCLUDEPATHs.

A couple of #include "mythconfig.h" needed changed to #include "config.h" to make picking up the configuration from the build directory easier.

If you are compiling in the same directory as the source, this change has no effect at all.

It has only been tested on x86. I can't see how it'd break other architectures, but it is probably worth mentioning.

Attachments

mythseperatebuild.diff Download (29.2 KB) - added by Dibblah 6 years ago.
mythseperatebuild2.diff Download (29.2 KB) - added by Dibblah 6 years ago.
Remove debug echo. No other changes.
mythseperatebuild3.diff Download (34.7 KB) - added by Dibblah 6 years ago.
Updated to SVN 9838, bugfix (don't recurse into created links), make sure all new / less used .pros have been updated
mythseperatebuild4.diff Download (36.0 KB) - added by Dibblah 6 years ago.
Updated to current (10047) SVN
mythseperatebuild5.diff Download (36.5 KB) - added by Dibblah 6 years ago.
Updated to 10224. Not ready for commit - svn version number doesn't work properly
mythseperatebuild6.diff Download (38.2 KB) - added by Dibblah 6 years ago.
Fixes previous bug with svnversion and themes not being installed

Change History

Changed 6 years ago by Dibblah

Changed 6 years ago by Dibblah

Remove debug echo. No other changes.

Changed 6 years ago by Dibblah

Updated to SVN 9838, bugfix (don't recurse into created links), make sure all new / less used .pros have been updated

Changed 6 years ago by Dibblah

Updated to current (10047) SVN

comment:1 Changed 6 years ago by danielk

  • Owner changed from ijr to danielk
  • Milestone changed from unknown to 0.20

comment:2 Changed 6 years ago by ijr

  • Owner changed from danielk to ijr

Was waiting on an updated patch before applying this..

Changed 6 years ago by Dibblah

Updated to 10224. Not ready for commit - svn version number doesn't work properly

Changed 6 years ago by Dibblah

Fixes previous bug with svnversion and themes not being installed

comment:3 Changed 6 years ago by ijr

  • Milestone changed from 0.20 to 0.21

comment:4 Changed 5 years ago by stuartm

  • Status changed from new to closed
  • Resolution set to wontfix

Closed at the request of the ticket reporter.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.