Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make mysql/mythtv support optional #1125

Closed
wants to merge 1 commit into from
Closed

Conversation

aballier
Copy link
Contributor

@aballier aballier commented Jul 6, 2012

Refs: http://trac.xbmc.org/ticket/11775
Authors: Zenkibou and vapier from that ticket.
Signed-off-by: Alexis Ballier aballier@gentoo.org

I've been using this for a while locally as I do not want nor need to install mysql. Comments ?

@aballier
Copy link
Contributor Author

aballier commented Jul 6, 2012

note also that this has been used for a while in gentoo too: http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/media-tv/xbmc/files/xbmc-9999-nomythtv.patch?view=log

@jmarshallnz
Copy link
Contributor

See related #1089 - @theuni - mind taking a look?

@aballier
Copy link
Contributor Author

aballier commented Jul 7, 2012

Hmm yes, this is almost the same..

should I scrap this one ? (this one has disable mythtv support too as was requested on the ticket)

http://trac.xbmc.org/ticket/11775
Based on a patch by Zenkibou and vapier from that ticket.
Signed-off-by: Alexis Ballier <aballier@gentoo.org>
@aballier
Copy link
Contributor Author

aballier commented Jul 7, 2012

updated with above comment and fixing bugs in the initial patch (wrong summary at end of configure, mythtv configure switch was useless as it was using mysql variables)

@theuni
Copy link
Contributor

theuni commented Jul 8, 2012

yes, #1089 is slated for this window, I plan to pull it in tomorrow. Thanks for the effort though :)

@theuni
Copy link
Contributor

theuni commented Jul 11, 2012

Same thing accomplished with 2d79579 from #1089. Thanks though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants