Apolyton Archive  |  Preserved copy of the Apolyton Civilization Site and its forums as they stood in September 2005. Read-only; nothing here can be posted to or replied to.  |  Forum index |  About this archive |  The 1998–2001 UBB forums
Today on Apolyton WARDELL INTERVIEW PROMO A.C.S. HISTORY CHAPTER 4 GET CIV4 /w FREE PLUS! A.C.S. PHOTO GALLERY GET A.O.M. V1.1
Apolyton Civilization Forums
main| civ2| civ3| civ4| smac| ctp2| ron| moo3| galciv| galciv2| alt| about|
ApolytonPLUS | register | search | faq | new posts | pm (-/-) | upload | members
hall of fame new! | civgroups | civgroups news | interviews | the column | radio | chat | directory | news | store | PLUS
Apolyton Civilization Forums : Powered by vBulletin version 2.0.3 Apolyton Civilization Forums > Call To Power II > CtP2-Source Code Project > #ifdef ACTIVISION_ORIGINAL
Show a Printable Version | Email This Page to Someone! | Receive updates to this thread | Report this to Apolyton news!
CivGroups
CTP2 Source Code Project (59): Not a Member - Join

bottom of page
  
Shall we continue using #ifdef ACTIVISION_ORIGINAL for tracking source code changes? (Time out:0 days after 08-07-2004, 20:57)
Yes!
No!
Let's start a discussion about that, first!
View Results [Edit Poll (moderators only)]
Author
Thread    < Last Thread     Next Thread > Post New Thread     Post A Reply
ctplinuxfan is offline ctplinuxfan
Warlord

Jan 2004
time: 06:36
  Old Post 08-07-2004 20:53
Edit/Delete Message Reply w/Quote
#1 Report this post to a moderator
#ifdef ACTIVISION_ORIGINAL Support Apolyton, buy Civilization 2

Hi,

Well, shall we continue using this directive, or not?

Pros:


  • you may see locations of changes (Alternatives: TortoiseMerge, WinMerge, etc.)
  • you may compile a binary based on apolytons code
  • it's the way we do it now


Cons:

  • fast growing sources due to duplicate code (original/changes)
  • you will not see changes of people not following this convention
  • this will lead to a binary based on slightly modified code
  • code may get harder to understand
  • changes may get harder to add
  • there's no similar change tracking for .ldl/.txt files
  • nested changes like system programming guarded by acitivision ifdefs might introduce bugs/strange behaviour due to complexity
  • changes can be localized by programs in Pros
  • a version control system aka VCS (like subversion, even more powerful than CVS) does the same, too, and much more


Any comment, additions to pros/cons list or opinions are welcome.
Happy Voting!

Solver is offline Solver
Apolyton Duke Of Something
Latvia, Riga
Sep 2000
time: 07:36
  Old Post 08-07-2004 21:26 Visit Solver<br><img src=/forums/images/staff-icon.gif>'s homepage!
Edit/Delete Message Reply w/Quote
#2 Report this post to a moderator
Tired of ads?

So far, while there are not THAT many canges to the code, I trust it's good. The main reasons are that it's easy to see where something was changed, and what was the original Activision way of doing it.

In perspective, when there's a positive lot of changed code, I think that simply commenting additions would be good, removing the original code. It's not exactly likely we'll need to compile anything with ACTIVISION_ORIGINAL on anyway, as we have just that in the shape of original ctp2.exe from our game disks.

Lui2 is offline Lui2
Chieftain
Saarbruecken
May 2004
time: 06:36
  Old Post 08-07-2004 21:52
Edit/Delete Message Reply w/Quote
#3 Report this post to a moderator
Suffering from ads?

I'm against using it longer, because the code is more difficult to understand the more changes in the code are included. And also I like a clean, small as possible source.

But there should be system, that tracks the changes and one can see step by step, what was changed. If we found a solution for that, it's a no brainer for me.

Until that, I use Filesync and WinDiff and a copy of the original code to see the differences between orinial and current source. So I can proof if Idid not have something wrong, with the "ACTIVISION_ORIGINAL" flag.

Solver is offline Solver
Apolyton Duke Of Something
Latvia, Riga
Sep 2000
time: 07:36
  Old Post 08-07-2004 22:03 Visit Solver<br><img src=/forums/images/staff-icon.gif>'s homepage!
Edit/Delete Message Reply w/Quote
#4 Report this post to a moderator
Inflate your Upload Space

Yes, but so far we have no CMS or VMS, and we won't have it soon at least here integrated on Apolyton, and using additional programs such as WinMerge takes additional time and resources. Our code isn't yet at the stage when there's a ton of #ifdefs - though, again, I will advocate a removal of those at some later point.

For instance, when we make the first 'public release' of CtP2:AE, it might be a good time to consider abolishing the ACTIVISION_ORIGINAL notation.

Flinx is offline Flinx
Prince
Toronto, ON CANADA
Nov 2001
time: 00:36
  Old Post 09-07-2004 01:02 Visit Flinx's homepage!
Edit/Delete Message Reply w/Quote
#5 Report this post to a moderator
Support Apolyton buy from Amazon

quote:
Originally posted by Solver
For instance, when we make the first 'public release' of CtP2:AE, it might be a good time to consider abolishing the ACTIVISION_ORIGINAL notation.


Are we not close to the goal of re-implementing all of the fixes in the 1.11 patch (the original goal of CTP2:AE v1.2 IIRC)?

kaan is offline kaan
Prince
Aarhus
Mar 2001
time: 05:36
  Old Post 09-07-2004 01:20
Edit/Delete Message Reply w/Quote
#6 Report this post to a moderator
Browse Apolyton AD-FREE

well there are many bugs left on that list, most of them relates to multiplayer though.

Flinx is offline Flinx
Prince
Toronto, ON CANADA
Nov 2001
time: 00:36
  Old Post 09-07-2004 02:12 Visit Flinx's homepage!
Edit/Delete Message Reply w/Quote
#7 Report this post to a moderator
Support Apolyton, buy Call to Power 2



do not play MP, completely forgot!

EPW is offline EPW
King
California, USA
Jan 2004
time: 21:36
  Old Post 09-07-2004 02:25 Visit EPW's homepage!
Edit/Delete Message Reply w/Quote
#8 Report this post to a moderator
Support Apolyton, buy Civilization: The Boardgame

MP isn't that critical at this point since the servers are down......

Solver is offline Solver
Apolyton Duke Of Something
Latvia, Riga
Sep 2000
time: 07:36
  Old Post 09-07-2004 02:29 Visit Solver<br><img src=/forums/images/staff-icon.gif>'s homepage!
Edit/Delete Message Reply w/Quote
#9 Report this post to a moderator
Support Apolyton, buy Galactic Civilizations

Servers are down, but there has been success in replacement methods of playing MP. Servers will also not be brought back up.

And yes, we're not close to re-implementing 1.1. All 1.1 SLIC and bugs are already fixed, I trust, but almost nothing relating to MP. And it's MP really where there's a ton of bugs - original MP code is barely functional at all, and that's what we are looking at now...

kaan is offline kaan
Prince
Aarhus
Mar 2001
time: 05:36
  Old Post 09-07-2004 04:25
Edit/Delete Message Reply w/Quote
#10 Report this post to a moderator
Lose 30 kilos (of popups)

We could set up a new server without too much trouble if we want to.
The anet library has gone open source (I think its LGPL).
There is a server program for download and plenty of information on the original authors homepage to make it happen.

heres a link:
http://www.kegel.com/anet/

EPW is offline EPW
King
California, USA
Jan 2004
time: 21:36
  Old Post 09-07-2004 04:26 Visit EPW's homepage!
Edit/Delete Message Reply w/Quote
#11 Report this post to a moderator
Support Apolyton or Terrorists Win

quote:
Originally posted by kaan
We could set up a new server without too much trouble if we want to.
The anet library has gone open source (I think its LGPL).
There is a server program for download and plenty of information on the original authors homepage to make it happen.

heres a link:
http://www.kegel.com/anet/


nice

kaan is offline kaan
Prince
Aarhus
Mar 2001
time: 05:36
  Old Post 09-07-2004 04:32
Edit/Delete Message Reply w/Quote
#12 Report this post to a moderator
Support Apolyton, pre-order Civilization IV

Come to think of it, I dont think it would be to hard to make an automatic ladder system with anet and a bit of technical skill.

look in the API documentation for score reporting, it looks very promising.

J Bytheway is offline J Bytheway
Emperor
England
Jul 2001
time: 05:36
  Old Post 10-07-2004 00:36 Visit J Bytheway's homepage!
Edit/Delete Message Reply w/Quote
#13 Report this post to a moderator
Support Apolyton, buy Call to Power 2

I'm still hoping we'll move to a CVS server (or similar) sometime soon, and then it would be a good idea to remove the ACTIVISION_ORIGINALed code, but in the mean time I think it's best left in. It's certainly not a good long term solution, though.

Fromafar is offline Fromafar
Prince

May 2003
time: 06:36
  Old Post 10-07-2004 23:48
Edit/Delete Message Reply w/Quote
#14 Report this post to a moderator
Got spare money?

If/when we do get some automated version control server ...

I would suggest storing the original Activision code as a baseline, and stripping all the ACTIVISION_ORIGINAL preprocessor directives before checking in the new code.

If not ...

We basically have to do the same. The current construct is starting to outlive its usefulness. It was intended to be able to find (and go to - another pro, you can search for it in the editor) changed pieces of code quickly. But this becomes less valid with each added change.

ctplinuxfan is offline ctplinuxfan
Warlord

Jan 2004
time: 06:36
  Old Post 14-07-2004 16:35
Edit/Delete Message Reply w/Quote
#15 Report this post to a moderator
Increase Your PM Length

Well, due to the results it seems we will keep this directive some time, until either CTP2:AE is finally released and/or some vcs has been setup for the main codeline.

This week i will start importing the mainstream into a local subversion repository, to get up 2 date with the sources. Unfortunately, i do not have all revisions of all changed files, because until now many links to patches have been removed, so there will be quite some ALL zips be used instead of a linear import of all changes to all files into the repository.

If nobody hasn't done until then, i will post the next ALL.zip to the Altered source files thread.

Afterwards i will start fixing cases of includes and merging some changes of the linux codeline into the main codeline, as long as they don't prevent the code from compiling and keep working under windows.
After that milestone the goal is to achieve a sync between the main code line, the linux port and perhaps Johannes Sixt's patch and remerge proceedings to the Altered source files thread and the linux port.

Btw: I can provide a dump of the subversion repository just in case we get that vcs later for the main codeline.

Martin Gühmann is offline Martin Gühmann
Emperor
Berlin, Germany
Mar 2001
time: 06:36
Post  Old Post 14-07-2004 22:20 Visit Martin Gühmann's homepage!
Edit/Delete Message Reply w/Quote
#16 Report this post to a moderator
Support Apolyton, buy Call to Power 2

Well I should have all of the mini files and of course the all patches, unfortunatly I had to do other things in the last weeks to upload the stuff to somewhere, I also have all the other all patches but not at the public place right now. I should have time this week end to upload the stuff.

Another thing with all these little patches is that you can easily forget one of these.

-Martin

ctplinuxfan is offline ctplinuxfan
Warlord

Jan 2004
time: 06:36
  Old Post 17-07-2004 03:13
Edit/Delete Message Reply w/Quote
#17 Report this post to a moderator
Inflate your Upload Space

no problem, i know what you are speaking of, i have been busy for quite some time, too (suppose that's why i haven't more files... )

well, i finally now know, from which posts the files are missing, and posted a list of the post dates here.
i also know that dealing with those files is rather time consuming , so it would also be ok if you decided to upload all (i will try to pick the right file from archive members timestamp/change text, then) or nothing at all.

  < Last Thread     Next Thread > Post New Thread     Post A Reply
All times are GMT. The time now is 05:36.
Apolyton Time is 00:36.
    top of page
Rate This Thread:
Forum Jump:
Forum Rules:
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts
HTML code is ON
vB code is ON
Smilies are ON
[IMG] code is ON
 




Contact Us - Apolyton Civilization Site - Support Us!

Building a better Apolyton through better information. Click here and take our poll!
Non-US visitors, click here!

Powered by: vBulletin Version 2.0.3
Copyright ©2000, 2001, Jelsoft Enterprises Limited.

Page generated in 0.0604 seconds (90.60% PHP - 9.40% MySQL) with 39 queries
Page Loading Time:

Support Apolyton: Amazon USA | Amazon UK | Amazon DE | Amazon FR |
Support Apolyton and get FREE PLUS, Buy from Chips&Bits: Galactic Civilizations | Galactic Civilizations: Deluxe Edition | Call to Power 2 | Civilization: The Boardgame | GURPS/ Alpha Centauri | Alpha Centauri | Civilization IV | Civilization III: Complete |


Front Page | Civilization IV | Civilization III | Civilization II | Call to Power II | Alpha Centauri | Master of Orion III
Rise of Nations | Galactic Civilizations | Galactic Civilizations II | Misc
Alt.Civs | Civ I | C:CtP I | About | News | Directory | Apolyton Store | Forums | Chat | Columns | Interviews | Newsletter
Scenario League | CSC | Clash of Civs | Spanish Site | CtP Maps | Cradle of Civ | WesW's Ctp1/2 Site | Civ3 Haven

apolyton.net | apolyton.com | civilization2.net | civilization3.net | civilization4.net | civilizationiv.info | calltopower.net | galciv.net | galciv2.net | moo3.net