Author Topic: CVS modifications..  (Read 3120 times)

0 Members and 1 Guest are viewing this topic.

Offline Joe

  • B&
  • Moderator
  • Hero Member
  • *****
  • Posts: 10319
  • In Soviet Russia, text read you!
    • View Profile
    • Github
CVS modifications..
« on: April 11, 2007, 05:03:16 pm »
Someone is making modifications to the CVS (notably clan features) and not noting them in javaop2/CHANGELOG.txt.

IF YOU MAKE A CHANGE, MAKE A NOTE OF IT, IF NOT IN THE CHANGELOG, ON THE FORUMS!

Thank you.
« Last Edit: April 11, 2007, 05:16:51 pm by Joe[x86/64] »
I'd personally do as Joe suggests

You might be right about that, Joe.


Offline Hdx

  • The Hdx!
  • Full Member
  • ***
  • Posts: 311
  • <3 Java/Cpp/VB/QB
    • View Profile
Re: CVS modifications..
« Reply #1 on: April 11, 2007, 05:15:18 pm »
Please note changes in the PLUGIN\ChangeLog.txt
This is one thing I will fight you forever on joe, JavaOp is modulirized. The CORE should stay the core, everything should be build off of Plugins as it is.
So there should not be 'JavaOp v###' when you update the BNetLogin, only when you update the Core.
~Hdx
http://img140.exs.cx/img140/6720/hdxnew6lb.gif
09/08/05 - Clan SBs @ USEast
 [19:59:04.000] <DeadHelp> We don't like customers.
 [19:59:05.922] <DeadHelp> They're assholes
 [19:59:08.094] <DeadHelp> And they're never right.

Offline Joe

  • B&
  • Moderator
  • Hero Member
  • *****
  • Posts: 10319
  • In Soviet Russia, text read you!
    • View Profile
    • Github
Re: CVS modifications..
« Reply #2 on: April 11, 2007, 05:19:01 pm »
Apparently you didn't commit your changelog, otherwise I wouldn't have been so pissed. :P

Or maybe I just missed it.

Either way, I moved your packet constants into javaop_pub.Constants.PacketConstants, where the rest of them are. Accordingly, I renamed 0x78 and 0x7A to their proper names in that list (previously UNKNOWN_78 and UNKNOWN_7A).

EDIT -
War3Clan
- bin
- - ClanConstants.class
- - PluginMain.class
- - PluginMain$Invite.class
- src
- - ClanConstants.java
- - PluginMain.java
- .classpath
- .project
- manifest.mf
- War3Clan.jardesc


No changelog on the CVS. :(

Also, please sign your commit comments.
« Last Edit: April 11, 2007, 05:23:21 pm by Joe[x86/64] »
I'd personally do as Joe suggests

You might be right about that, Joe.


Offline Hdx

  • The Hdx!
  • Full Member
  • ***
  • Posts: 311
  • <3 Java/Cpp/VB/QB
    • View Profile
Re: CVS modifications..
« Reply #3 on: April 11, 2007, 05:22:58 pm »
I haven't upload my new War3 plugin 0.o
maybe someone else did.
Last thing I uploaded was the eintire database. <3 recursive batch scripts.
~Hdx
« Last Edit: April 11, 2007, 05:27:30 pm by HdxBmx27 »
http://img140.exs.cx/img140/6720/hdxnew6lb.gif
09/08/05 - Clan SBs @ USEast
 [19:59:04.000] <DeadHelp> We don't like customers.
 [19:59:05.922] <DeadHelp> They're assholes
 [19:59:08.094] <DeadHelp> And they're never right.

Offline chuck

  • Full Member
  • ***
  • Posts: 335
  • Canadian Biathlete
    • View Profile
    • Chucks Blog
Re: CVS modifications..
« Reply #4 on: April 12, 2007, 12:54:22 pm »
Since both of you are almost always comiting changes, you should have your own CVS accounts. cvsuser should be for the casual developer, not the main developers!
Chucks Blog
JavaOp2 Plugins

Quote
Error, keyboard not connected. Press F1 to continue.

Offline Hdx

  • The Hdx!
  • Full Member
  • ***
  • Posts: 311
  • <3 Java/Cpp/VB/QB
    • View Profile
Re: CVS modifications..
« Reply #5 on: April 12, 2007, 04:24:55 pm »
We do have our own accounts.
Code: [Select]
  -Advertisements
    -Removed debug code.
    -Added 'Download ads', automatically download the add files from Bnet's FTP server if you do not have it locally.
  -Clan
    -Changed to 'WhisperOnLeave', Why the hell was it named Clan?
*committing the changes soon*
I'm going through the plugins making mods and cleaning it up.

~Hdx
http://img140.exs.cx/img140/6720/hdxnew6lb.gif
09/08/05 - Clan SBs @ USEast
 [19:59:04.000] <DeadHelp> We don't like customers.
 [19:59:05.922] <DeadHelp> They're assholes
 [19:59:08.094] <DeadHelp> And they're never right.

Offline chuck

  • Full Member
  • ***
  • Posts: 335
  • Canadian Biathlete
    • View Profile
    • Chucks Blog
Re: CVS modifications..
« Reply #6 on: April 13, 2007, 12:01:17 pm »
We do have our own accounts.
Oh, ok.

By the way, what ever happened to www.javaop.com? It looks like it has not been updated in awhile.
Chucks Blog
JavaOp2 Plugins

Quote
Error, keyboard not connected. Press F1 to continue.

Offline Joe

  • B&
  • Moderator
  • Hero Member
  • *****
  • Posts: 10319
  • In Soviet Russia, text read you!
    • View Profile
    • Github
Re: CVS modifications..
« Reply #7 on: April 14, 2007, 12:33:46 am »
iago owns that.. I could probably get a hold of access to it, but I don't feel like it. :P
I'd personally do as Joe suggests

You might be right about that, Joe.