MinetestForFun / server-minetestforfun

Repository of the subgame and mods of "MinetestForFun" server

https://www.xorhub.com

The Unlicense

27 stars 10 forks source link

Quick hack : no friction, no mesecon, rail_power=accelerate, rail_bra… #493

Closed Coethium closed 8 years ago

Coethium commented 8 years ago

…ke=brake

492

ghost commented 8 years ago

J'ai survolé vite fait le code et il y a 2 minetest.chat_send_all() à enlevé lignes 428 et 554.

il faudrait virer l'abm à la fin du fichier, car elle repose les nodes en continue, on devait le faire y a quelques mois déjà mais on a oublié :).

Coethium commented 8 years ago

La boulette ;-) Corrigé

BetterToAutomateTheWorld commented 8 years ago

+1 pour le merge suite au recommandations de @crabman77

Merci pour ton travail @Coethium

Coethium commented 8 years ago

Avec plaisir ! Est-ce que j'attaque l'optimisation ? (voir #492 )

PS: je ne sais pas où le dire, dans MFF Classic sous le spawn en 42,-58,1 j'ai installé une station de métro, donc quelqu'un ayant les droits area au dessus peut y miner un accès (suggestion : par assenseur à eau). Pour l'instant il n'y a qu'une seule ligne d'environ 400blocks vers le sud.

Lymkwi commented 8 years ago

Hormis ce que j'ai mis, et toute les gros blocs commentés qu'il vaudrait mieux retirer (ça aère le code), ça me va aussi.

BetterToAutomateTheWorld commented 8 years ago

@Coethium Je te passe les droits sur l'area ce soir (si je n'oubli pas), tu pourra miner et construire une maison sur l'île du spawn ;)

Coethium commented 8 years ago

Wow, je n'en demandais pas tant ! Mais merci :-)

BetterToAutomateTheWorld commented 8 years ago

Tu es ajouté à l'area :

Coethium commented 8 years ago

Encore merci, j'entame l'accès

Lymkwi commented 8 years ago

Après mes tests tout fonctionne. La gravité influence encore les carts, et il y a une erreur que j'ai mentionné plus haut; À fixer, et :+1:

Coethium commented 8 years ago

Corrigé, je t'ai donné l'accès au repo car sur des petits trucs comme ça c'est dommage d'attendre.

PS: une fois que ce sera mergé, si je veux proposer de nouvelles modifs, je le fais dans cette branche ? ou j'en fork une nouvelle ?

Lymkwi commented 8 years ago

Tu peux les mettre dans la même

Lymkwi commented 8 years ago

e0a3d098f55e634153b5236a0e7922fa79cd1701 2f7e94dbfd140f3c9beb82984380c67746a67102 5cc518badf24fa8480c948d22e4bd3d32feebe3b 1e6f5424ca5f7a8806e0466351462611a6278fd6

Coethium commented 8 years ago

Attention, encore un fix : il restait deux appels à cart: dans rails.lua, du coup crash server comme l'avait relevé LeMagnésium :/

BetterToAutomateTheWorld commented 8 years ago

Quelqu'un peut confirmer que https://github.com/MinetestForFun/server-minetestforfun/commit/66ab74194d5270b2ba8dd028aead6c7f9a661d50 fix les crashes ?

Après validation je merge

ghost commented 8 years ago

crash fixé https://github.com/MinetestForFun/server-minetestforfun/commit/0496cd14ebae92235f588e41621c510d62570683