| Summary: | games-strategy/megaglest-3.13.0 version bump | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Andrei Slavoiu <ansla80> |
| Component: | Current packages | Assignee: | Gentoo Games <games> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | andrius, esigra, pacho |
| Priority: | Normal | Keywords: | PMASKED |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Bug Depends on: | |||
| Bug Blocks: | 574082, 600812, 614408 | ||
| Deadline: | 2020-11-23 | ||
| Attachments: | megaglest-ebuild.patch | ||
Is this going to be bumped? If not then megaglest should be removed from the tree. It's already more than half a year that this ebuild fails to compile. (In reply to Andrius Štikonas from comment #1) > Is this going to be bumped? > > If not then megaglest should be removed from the tree. It's already more > than half a year that this ebuild fails to compile. Games team is understaffed. If you want things done faster then please join. the ebuild would need to stop using games.eclass, use regular PATHS like regular packages and try to bump to eapi6 if possible Thanks |
Created attachment 468112 [details, diff] megaglest-ebuild.patch It was released 2 weeks ago and is a higher priority than usual as current version in portage does not build with miniupnpc-2 (I can log a bug about that if needed). Attached patch was tested to build and run correctly with the following USE flags: "streflop tools unicode -debug -editor -fribidi -model-viewer -static -videos -wxuniversal". For megaglest-data just renaming the ebuild was enough. Now some explanations about the patch. I removed the patches because they did not apply cleanly and they seamed unnecesary anymore. The subtitutions "${GAMES_BINDIR#/usr/}" and "${GAMES_DATADIR#/usr/}/${PN}" are meant to turn the paths into ones relative to ${PREFIX} as the build system expects them. I have no idea how to do proper substion of ${PREFIX} so I hardcoded /usr/. Once doing that it is no longer required to override CUSTOM_DATA_INSTALL_PATH as it will be generated correctly. Also, I removed the doins for the png file because it no longer exists. Apparently an ico file is installed now.