Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 492946

Summary: games-action/minetest_game-0.4.7 pulling in minetest_common-0.4.6 as dependency
Product: Gentoo Linux Reporter: Account Disabled (uid 86422) <account-disabled-uid-86422>
Component: [OLD] GamesAssignee: Gentoo Games <games>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: Patch that removes minetest_common as dependency of minetest_game

Description Account Disabled (uid 86422) 2013-11-30 14:38:54 UTC
Pulling in games-action/minetest_common-0.4.6 as dependency for games-action/minetest_game-0.4.7 is deprecated by upstream.

Reproducible: Always

Steps to Reproduce:
1. emerge -v =games-action/minetest_game-0.4.7

Actual Results:  
Pulls in games-action/minetest_common-0.4.6

Expected Results:  
Don't pull in games-action/minetest_common-0.4.6

Like in case of games-action/minetest_game-0.4.8, pulling in games-action/minetest_common is deprecated by upstream. The split of the game data into
common/build/survival was introduced with minetest-0.4.6 and dropped with minetest-0.4.7. But dropping minetest_{common,build,survival}-0.4.6 doesnt have to happen right now, as atleast minetest-0.4.7 has still support for common/build/survival as some mods MAY require these packages. So common/build/survival should be treated as optional and it may be a good idea to mention it in the elog of games-action/minetest.
Comment 1 Account Disabled (uid 86422) 2013-11-30 15:55:09 UTC
Talked with upstream about mods and their dependency to common/build/survival.
All activly developed mods are ported back to the old behaviour of using only minetest_game. So dropping common/build/survival should be safe now.
Comment 2 Julian Ospald 2013-12-04 21:06:09 UTC
attach a patch to the ebuild
Comment 3 Account Disabled (uid 86422) 2013-12-05 16:54:26 UTC
Working on it. Still have to sort out a proper workflow for me.

(In reply to Julian Ospald (hasufell) from comment #2)
> attach a patch to the ebuild
Comment 4 Account Disabled (uid 86422) 2013-12-11 13:09:40 UTC
Created attachment 365100 [details, diff]
Patch that removes minetest_common as dependency of minetest_game

As i am new to things like creating patches, i would like to get comments, hints, criticism about this topic(especially if i made mistakes).
Comment 5 Julian Ospald 2013-12-11 20:42:24 UTC
+  11 Dec 2013; Julian Ospald <hasufell@gentoo.org> minetest_game-0.4.7.ebuild:
+  fix deps wrt #492946