Mechanized Assault & eXploration Reloaded



#11 Jan 07, 2014 2:02 pm
nonsinn nonsinn Offline
G-Mod, Approved Member
Registered since: Apr 02, 2012
Posts: 254


Subject: Re: ResInstaller compilieren
Klarstellung:
a) es sieht aus als hätte ObeY das erkannt, dass das mit den Optionen nicht vorhanden ist / er falsch verstanden hat. ⇒ #653 (resinstaller option)

b) Die Integration des Resinstallers in die install.exe von 0.2.8 funktioniert nicht ⇒ #642 (resinstaller during install routine 0.2.8. not working)

c) schön das du nun den Resinstaller auch compilieren kannst… und nun ab ans Ticket #653 :P
[15:23:19] nonsinn [..] heißt nun bug-hunter
[19:58:09] Eiko_ [..] heißt nun mother-of-bugs
[20:38:11] <bug-hunter> mother-of-bugs: ..argh..
[20:38:26] <@mother-of-bugs> want some new bugs?
↑  ↓

#12 Jan 07, 2014 4:13 pm
ObeY Offline
Approved Member
Registered since: Aug 12, 2013
Posts: 52


Subject: Re: ResInstaller compilieren
Quote by nonsinn:
c) schön das du nun den Resinstaller auch compilieren kannst… und nun ab ans Ticket #653 :P
jawohl! :-)
↑  ↓

#13 Jan 27, 2014 6:30 pm
ObeY Offline
Approved Member
Registered since: Aug 12, 2013
Posts: 52


Subject: Re: ResInstaller compilieren
ich hab ne option für die neuen grafiken zu verwenden in den resinstaller reingebracht(inkl argv[4], ich vermute das ist dann für den maxr installer).
wie läuft das nun mit dem einchecken des codes?
soll ich jemanden die resinstaller.cpp zukommen lassen damit ihr mal prüfen könnt ob ich nicht einfach den code verhunzt habe ;-)

zweite frage:
ich habe versucht maxr zu compilieren, habe die sdl libs aus der erklärung genommen(1.2) aber das compilieren schlägt fehlt (ewig viele fehlermeldungen - kann ich bei bedarf nachreichen).
dann habe ich zusätzlich SDL2 als lib hinzugefügt. danach waren es nur noch eine handvoll fehlermeldungen aber ging trotzdem nicht
werden in der erklärung zum compilieren die falschen libs aufgeführt?
↑  ↓

#14 Jan 28, 2014 2:02 pm
alzi alzi Offline
Developer, Moderator
Registered since: Aug 12, 2007
Posts: 339


Subject: Re: ResInstaller compilieren
Hallo ObeY,

erstell am besten ein .patch-File mit den Änderungen an der resintaller.cpp und häng sie an das entsprechende Ticket an. Dann kann sich das ein Entwickler anschauen. Falls dir das mit dem .patch-File jetzt nichts sagt, kannst du auch einfach die ganze resinstaller.cpp anhängen.

Zum bauen von maxr selbst und SDL: der trunk im SVN ist erst vor etwa 2 Wochen auf SDL2 umgestellt worden. Du hast also recht, dass die Anleitung jetzt nicht mehr stimmt. Es müssen die SDL2 und nicht die alten SDL Bibliotheken verwendet werden.
Dann sollte es aber denke ich gehen (wobei ich das jetzt auch noch nicht getestet habe, seit es eingecheckt wurde). Kannst du die Fehlermeldungen hier posten?

edit: habe es gerade nochmal getestet und zwei kleine Änderungen eingecheckt. Jetzt kompiliert es zumindest bei mir mit VS2013 und SDL2 wieder.
Albert Ziegenhagel
This post has been edited 1 times. Last edit on Jan 28, 2014 2:32 pm by alzi. ↑  ↓

#15 Jan 28, 2014 10:35 pm
ObeY Offline
Approved Member
Registered since: Aug 12, 2013
Posts: 52


Subject: Re: ResInstaller compilieren
habe das mit der erzeugung des patch files im ankh svn jetzt nicht ad hoc gefunden
https://trac.maxr.org/ticket/653

meine änderung habe ich im detail im ticket beschrieben

habe mir gerade auch den aktuellen maxr code geholt und versuche es morgen mit vs 2010 und sdl2 zu compilieren
zur not nehm ich mal 2012 oder besorg mir 2013 auf der arbeit

wenn ihr mit meiner Änderung einverstanden seid würde ich gerne weitere Sachen machen.
↑  ↓

#16 Jan 29, 2014 4:13 pm
alzi alzi Offline
Developer, Moderator
Registered since: Aug 12, 2007
Posts: 339


Subject: Re: ResInstaller compilieren
Erstmal Danke für den Patch!

Habe die Änderungen mal durchgesehen und es sieht gut aus und kompiliert soweit! Testen konnte ich es zwar noch nicht, aber das sollte schon stimmen.

Nur zwei kleine Anmerkungen:
- Im Code bitte auch die Kommentare auf Englisch. Wir haben mittlerweile Entwickler, die nicht deutschsprachig sind.
- Achte darauf, die gleiche Formatierung wir im Rest des Codes zu verwenden. z.B hast du zum Einrücken zwei Leerzeichen verwendet, sonst wurden im Code aber überall Tabs verwendet oder sind im Rest des Codes z.B. auch die geschweiften Klammern in einer eigenen Zeile, du hattest das anders. Hier kann man sich natürlich ewig drüber streiten was besser und was schlechter ist, darum geht es aber egtl. gar nicht. Nur eines ist wichtig: einheitlich muss es sein! Irgendwo im Forum gibt es glaube ich einen Thread wo ein paar Regeln definiert wurden. Ob die noch eingehalten werden, wage ich aber zu bezweifeln. Am besten einfach schauen, wie der Code so formatiert ist und einfach genau so machen Wink

Auf jeden Fall habe ich die paar Anmerkungen behoben und das Ganze eingecheckt!

Über weitere Patches würden wir uns natürlich freuen Smiling

Dann noch zum kompilieren von maxr:
Das sollte definitiv auch (noch) mit VS2010 und VS2012 gehen. Wenn Fehler kommen, poste sie bitte hier. Ich kann das leider nur nicht mehr testen, weil ich nur noch VS2013 installiert habe. Trotzdem kann ich dir nur empfehlen auf die neuen Versionen zu upgraden, wenn du die Möglichkeit hast (z.B. bei der Express Edition). Bedenke, dass du dann aber ggf. etwaige Bibliotheken, die du selbst gebaut hast, neu bauen musst.
Albert Ziegenhagel
↑  ↓

#17 Jan 29, 2014 4:38 pm
ObeY Offline
Approved Member
Registered since: Aug 12, 2013
Posts: 52


Subject: Re: ResInstaller compilieren
Wir haben hier auf der Arbeit auch unseren StyleGuide beim coden (z.B. keine Tabs nur 2 Leerzeichen pro Einrückung und die geschweiften Klammern nicht in einer extra Zeile :D)
Daher habe ich kein Problem damit mich an eure Konventionen zu halten und werde als letzter eine Diskussion anfangen ;-)
English geht klar (wobei mir auch einige deutsche Kommentare im MAXR code aufgefallen sind)
StyleGuide habe ich hier nicht beim überfliegen und suchen gefunden aber ich denke das ergibt sich dann wie du gesagt hast aus dem Code.

Ich kann mir hier sicher VS 2013 besorgen (2012 habe ich auch, nur nicht installiert) werde aber erst 2010 testen und eine Rückmeldung geben.
Würde dann gerne als nächstes die Kompilierungsanleitung auf den neuesten Stand bringen (und um den ResInstaller erweitern - die habe ich schon fertig).

Auch nervt mich das Verhalten des Gutachters im Auto Modus. Das ist ne Katastrophe! Aber eine richtige Lösung habe ich noch nicht die man Umsetzung könnte. Ansonsten schaue ich mal durch die Tickets und wenn ich was finde was mein Können nicht übersteigt gehe ichs an. (bin eigentlich ein Pascal Entwickler mit einigen Ausflügen in C#)
↑  ↓

#18 Jan 29, 2014 5:02 pm
alzi alzi Offline
Developer, Moderator
Registered since: Aug 12, 2007
Posts: 339


Subject: Re: ResInstaller compilieren
Quote by ObeY:
English geht klar (wobei mir auch einige deutsche Kommentare im MAXR code aufgefallen sind)

Jap, aus alten Zeiten sind da leider noch einige drinnen. Auch manche Variablennamen sind noch deutsch (daran erkennt man z.B. auch, dass du hier eine Codestelle gefunden hast, die _wirklich_ alt ist. Betreten auf eigene Gefahr Wink )

Quote by ObeY:
StyleGuide habe ich hier nicht beim überfliegen und suchen gefunden aber ich denke das ergibt sich dann wie du gesagt hast aus dem Code.

Kann auch sein, dass das nur in der Entwicklerecke des Forums zu finden ist (auf die du vermutlich keinen Zugriff hast?!). Wie gesagt, würd ich mich drauf aber auch nicht mehr verlassen.

Quote by ObeY:
Würde dann gerne als nächstes die Kompilierungsanleitung auf den neuesten Stand bringen (und um den ResInstaller erweitern - die habe ich schon fertig).

Das wäre spitze!

Quote by ObeY:
Auch nervt mich das Verhalten des Gutachters im Auto Modus. Das ist ne Katastrophe! Aber eine richtige Lösung habe ich noch nicht die man Umsetzung könnte. Ansonsten schaue ich mal durch die Tickets und wenn ich was finde was mein Können nicht übersteigt gehe ichs an. (bin eigentlich ein Pascal Entwickler mit einigen Ausflügen in C#)

Kann gut sein, dass es zur Gutachter AI sogar schon ein Ticket gibt. Ich kann mich zumindest dunkel dran erinnern, dass da schon mal irgendwo drüber diskutiert wurde. Ich begutachte für gewöhnlich am Anfang immer selbst und lass die Dinger dann später einfach machen.
Ansonsten bei C++ spezifischen Sachen oder auch sonst Unverständnissen im Code, gerne einfach Fragen. Du wirst aber schnell merken, dass der maxr Code nicht gerade ein Meisterwerk ist. Man muss hier bedenken, dass die meisten Entwickler (inklusive mir) damals an maxr ihre ersten Programmiererfahrungen an einem größeren Projekt gesammelt haben. Trotzdem ist es meiner Meinung nach eine deutliche Verbesserung zu dem, worauf wir aufgesetzt hatten.
Albert Ziegenhagel
↑  ↓

#19 Jan 29, 2014 9:58 pm
ObeY Offline
Approved Member
Registered since: Aug 12, 2013
Posts: 52


Subject: Re: ResInstaller compilieren
Quote by alzi:

Kann auch sein, dass das nur in der Entwicklerecke des Forums zu finden ist (auf die du vermutlich keinen Zugriff hast?!). Wie gesagt, würd ich mich drauf aber auch nicht mehr verlassen.
kenn ich nicht. wenn der zeitpunkt gekommen ist, würde ich mich freuen da lesen und schreiben zu können

Quote by alzi:
Kann gut sein, dass es zur Gutachter AI sogar schon ein Ticket gibt. Ich kann mich zumindest dunkel dran erinnern, dass da schon mal irgendwo drüber diskutiert wurde. Ich begutachte für gewöhnlich am Anfang immer selbst und lass die Dinger dann später einfach machen.
ja so mache ich es auch und eine AI wird niemals an die methode rankommen die jeder für sich so hat (oder halt exakt nach einer methode die dann aber nicht unbedingt allgemeingültig ist)
ich war auch schon in der stelle im code und habe mich gewundert wieviele parameter schon jetzt berücksichtigt werden (nähe zur eigenen basis usw)

Quote by alzi:
Ansonsten bei C++ spezifischen Sachen oder auch sonst Unverständnissen im Code, gerne einfach Fragen. Du wirst aber schnell merken, dass der maxr Code nicht gerade ein Meisterwerk ist. Man muss hier bedenken, dass die meisten Entwickler (inklusive mir) damals an maxr ihre ersten Programmiererfahrungen an einem größeren Projekt gesammelt haben. Trotzdem ist es meiner Meinung nach eine deutliche Verbesserung zu dem, worauf wir aufgesetzt hatten.
danke für das angebot. darauf komme ich gerne zurück wenn ich selber absolut nicht mehr klar komme.
es gehört aber auch ein wenig fleiß dazu. möchte euch nicht wegen jeder kleinigkeit nerven und euch mehr arbeit machen als ich beisteuern kann ...

vielen dank fürs die ganzen infos!
↑  ↓

#20 Jan 30, 2014 12:06 am
beko beko Offline
Developer, Administrator
Registered since: Jun 04, 2004
Posts: 2,279


Subject: Re: ResInstaller compilieren
> würde ich mich freuen da lesen und schreiben zu können

fixed
Bernd Kosmahl
"Sir, we are surrounded!" - "Great - we can attack in any direction."
↑  ↓

Pages (3): 1, 2, 3


All times are GMT +01:00. Current time: 8:40 pm.