Created attachment 322408 [details] tvheadend-3.0 ebuild The ebuild for 3.0 is based off my git ebuild from bug #288022. I've updated the homepage and src uri. The version number is set manually in src_prepare since upstream source only uses git for versioning.
Please base any version bumps on the ebuild already in portage. You may also want to use vcs-snapshot eclass.
(In reply to comment #1) > Please base any version bumps on the ebuild already in portage. You may also > want to use vcs-snapshot eclass. The git ebuild only differs from 2.12 to reflect the changes upstream made post 2.12 release. The 3.0 ebuild only adds what I mentioned in comment 1. Thanks for the tip about vcs-snapshot eclass. The hash that is appended to the unpacked directory name is very annoying. I will post an updated ebuild.
Created attachment 322410 [details] tvheadend-3.0 ebuild
Created attachment 326666 [details] tvheadend-3.0 ebuild Updated to include fixes for bugs #438426 and #438424.
Created attachment 326682 [details] tvheadend-3.0 ebuild Improved version of tvheadend ebuild: - remove redundant mandir and datadir override; - add initial prefix support; - add missing zlib dependency, remove automagic on zlib, reorganize dependencies; - simplify src_install, add DOCS variable.
(In reply to comment #5) > - remove redundant mandir and datadir override; Without the mandir override the manual gets installed directly in /usr/share/man and doesn't work for me. I added the datadir override to prevent pollution of /usr/share. This was in response to comment 44 in bug #288022.
(In reply to comment #6) > (In reply to comment #5) > > - remove redundant mandir and datadir override; > > Without the mandir override the manual gets installed directly in > /usr/share/man and doesn't work for me. > > I added the datadir override to prevent pollution of /usr/share. This was in > response to comment 44 in bug #288022. Yep, it seems that you are right - configure script is broken and will not work as upstream says.
Created attachment 326684 [details] tvheadend-3.0 ebuild All needed paths are returned to src_configure stage, cause path construction in configure script is broken. Also, please note, that mmx and sse configure flag handling broken and thus should be fixed(see configure script for details)
We may need a new proxy maintainer for this package. Leho Kraav hasn't responded to the last 2-3 tvheadend bugs.
Yep, I can't keep up at all at the moment.
(In reply to comment #10) > Yep, I can't keep up at all at the moment. No worries. Anyone interested to pick up this package? BT I see you are doing most of the work here so you seem to be a good candidate :)
(In reply to comment #11) > No worries. Anyone interested to pick up this package? BT I see you are > doing most of the work here so you seem to be a good candidate :) Unfortunately I can't do it. Sorry.
Created attachment 327386 [details] ebuild modified for 3.2 I modified the 3.0 ebuild to work with 3.2, which has been released a few days ago. Will try to fix it up a bit more later.
(In reply to comment #13) I have some feedback regarding the ebuild. 1. SRC_URI should be: https://github.com/downloads/${PN}/${PN}/${P}.tar.gz 2. You should explicitly disable dvbscan. When enabled the package will fetch the latest dvb-scan files during configure, which is against ebuild rules from what I understand. 3. The support/version script will now parse the debian/changelog file for the version number. However the version number is incorrectly set to 3.1.694 so we still need to set the version number manually.
Created attachment 327402 [details] tvheadend-3.2.ebuild 1. Done. I couldn't figure out the ${P} 2. Done. 3. Maybe done. Not sure if the fix is correct.
Comment on attachment 327402 [details] tvheadend-3.2.ebuild Little notice on SRC_URI - you should use "mirror://github/${PN}/${P}.tar.gz"
(In reply to comment #16) > Comment on attachment 327402 [details] > tvheadend-3.2.ebuild > > Little notice on SRC_URI - you should use "mirror://github/${PN}/${P}.tar.gz" Sorry, i mean "mirror://github/${PN}/${PN}/${P}.tar.gz"
You can also make zlib support optional since it can be controlled with configure now.
Created attachment 327508 [details] tvheadend-3.2 ebuild Minor update of 3.2 ebuild. - Add zlib use flag. - Simplify setting version number sed. - Sync removing '-Werror' sed with 2.12 ebuild.
(In reply to comment #17) > Sorry, i mean "mirror://github/${PN}/${PN}/${P}.tar.gz" This doesn't seem to work for packages that are not in the tree.
(In reply to comment #19) > Created attachment 327508 [details] > tvheadend-3.2 ebuild > > Minor update of 3.2 ebuild. > > - Add zlib use flag. > - Simplify setting version number sed. > - Sync removing '-Werror' sed with 2.12 ebuild. I've committed this, with some minor edits. Please report if there are any problems.
