--- commands.py +++ commands.py @@ -58,6 +58,8 @@ def reallyrmtree(path): + if os.path.isfile(os.path.join(path, 'asm-4.1.tar.gz')): #Check if asm exists, indicating the libs folder, if so, don't delete it + return if not sys.platform.startswith('win'): if os.path.exists(path): shutil.rmtree(path) @@ -139,6 +141,8 @@ fieldnames = csvreader.fieldnames return set(fieldnames) +def commands_sanity_check(): + print 'Commands patch applied successfully' class Commands(object): """Contains the commands and initialisation for a full mcp run""" @@ -194,6 +198,7 @@ sys.exit(1) self.checkjava() + self.checkscala() self.readcommands(verify, no_patch=no_patch) def checkcommand(self, name, command, java=False, single_line=False, check_return=True, error=True): @@ -230,6 +235,8 @@ self.checkcommand('java', '%s -version' % self.cmdjava) self.checkcommand('javac', '%s -version' % self.cmdjavac) self.checkcommand('javac runtime', '%s -J-version' % self.cmdjavac) + if self.cmdscalac: self.checkcommand('scalac', '%s -version' % self.cmdscalac) + if self.cmdscalac: self.checkcommand('scalac runtime', '%s -J-version' % self.cmdscalac) self.checkcommand('retroguard', '%s --version' % self.retroguard, java=True) self.exceptor = os.path.normpath(self.config.get('COMMANDS', 'Exceptor')) @@ -350,6 +357,8 @@ self.cmdfernflower = self.config.get('COMMANDS', 'CmdFernflower', raw=1) % (self.cmdjava, self.fernflower) self.cmdexceptor = self.config.get('COMMANDS', 'CmdExceptor', raw=1) % (self.cmdjava, self.exceptor) self.cmdrecomp = self.config.get('COMMANDS', 'CmdRecomp', raw=1) % self.cmdjavac + if self.cmdscalac: self.cmdrecompscalac = self.config.get('COMMANDS', 'CmdRecompScalac', raw=1) % self.cmdscalac + else: self.cmdrecompscalac = None self.cmdstartsrv = self.config.get('COMMANDS', 'CmdStartSrv', raw=1) % self.cmdjava self.cmdstartclt = self.config.get('COMMANDS', 'CmdStartClt', raw=1) % self.cmdjava @@ -753,6 +762,36 @@ self.cmdjavac = '"%s"' % os.path.join(results[0], 'javac') self.cmdjava = '"%s"' % os.path.join(results[0], 'java') + def checkscala(self): + results = [] + if not results: + try: + # TODO: Verify at least version 2.10 + self.runcmd('scalac -version', quiet=True) + results.append('') + except (CalledProcessError, OSError): + pass + if not results: # Stupid windows... + try: + # TODO: Verify at least version 2.10 + self.runcmd('scalac.bat -version', quiet=True) + results.append('') + except (CalledProcessError, OSError): + pass + try: + self.runcmd('scala -target:jvm-1.6 -version', quiet=True) + results.append('') + except (CalledProcessError, OSError): + self.logger.info('"scala" does not support jvm-1.6 target, it is out of date. Ignoring') + results = [] + self.cmdscalac = None + else: + if not results: + self.logger.warning('"scalac" is not found on the PATH. Scala files will not be recompiled') + self.cmdscalac = None + else: + self.cmdscalac = '"%s"' % os.path.join(results[0], 'scalac') + def checkjars(self, side): jarlk = {CLIENT: self.jarclient, SERVER: self.jarserver} md5jarlk = {CLIENT: self.md5jarclt, SERVER: self.md5jarsrv} @@ -798,6 +837,8 @@ binlk = {CLIENT: self.binclient, SERVER: self.binserver} testlk = {CLIENT: self.testclient, SERVER: self.testserver} + if side == SERVER and not os.path.exists(os.path.join(binlk[side], os.path.normpath(testlk[side] + '.class'))): + return self.checkbins(CLIENT) if not os.path.exists(os.path.join(binlk[side], os.path.normpath(testlk[side] + '.class'))): return False return True @@ -1038,6 +1079,9 @@ pathbinlk = {CLIENT: self.binclient, SERVER: self.binserver} pathsrclk = {CLIENT: self.srcclient, SERVER: self.srcserver} pathlog = {CLIENT: self.clientrecomplog, SERVER: self.serverrecomplog} + + if side == SERVER: #Warn about server because FML recombines it into client + self.logger.info('Server side recompiling might be skipped, this is normal, do not complain to FML/Forge team about server side.') if not os.path.exists(pathbinlk[side]): os.makedirs(pathbinlk[side]) @@ -1051,6 +1095,29 @@ all_files = True append_pattern = False pkglist = filterdirs(pathsrclk[side], '*.java', append_pattern=append_pattern, all_files=all_files) + if self.cmdrecompscalac: # Compile scala before java as scala scans java files, but not vice-versa + pkglistscala = pkglist[:] + pkglistscala.extend(filterdirs(pathsrclk[side], '*.scala', append_pattern=append_pattern, all_files=all_files)) + dirs = ' '.join(pkglistscala) + classpath = os.pathsep.join(cplk[side]) + forkcmd = self.cmdrecompscalac.format(classpath=classpath, sourcepath=pathsrclk[side], outpath=pathbinlk[side], pkgs=dirs) + try: + self.runcmd(forkcmd, log_file=pathlog[side]) + except CalledProcessError as ex: + self.logger.error('') + self.logger.error('== ERRORS FOUND in SCALA CODE ==') + self.logger.error('') + for line in ex.output.splitlines(): + if line.strip(): + if line.find("jvm-1.6") != -1: + self.logger.error(' === Your scala version is out of date, update to at least 2.10.0 ===') + if line[0] != '[' and line[0:4] != 'Note': + self.logger.error(line) + if '^' in line: + self.logger.error('') + self.logger.error('================================') + self.logger.error('') + raise dirs = ' '.join(pkglist) classpath = os.pathsep.join(cplk[side]) forkcmd = self.cmdrecomp.format(classpath=classpath, sourcepath=pathsrclk[side], outpath=pathbinlk[side], @@ -1059,7 +1126,7 @@ self.runcmd(forkcmd, log_file=pathlog[side]) except CalledProcessError as ex: self.logger.error('') - self.logger.error('== ERRORS FOUND ==') + self.logger.error('== ERRORS FOUND in JAVA CODE ==') self.logger.error('') for line in ex.output.splitlines(): if line.strip(): @@ -1072,7 +1139,7 @@ raise def startserver(self): - classpath = [self.binserver] + self.cpathserver + classpath = [self.binclient, self.srcclient] + self.cpathserver classpath = [os.path.join('..', p) for p in classpath] classpath = os.pathsep.join(classpath) os.chdir(self.dirjars) @@ -1080,7 +1147,7 @@ self.runmc(forkcmd) def startclient(self): - classpath = [self.binclient] + self.cpathclient + classpath = [self.binclient, self.srcclient] + self.cpathclient classpath = [os.path.join('..', p) for p in classpath] classpath = os.pathsep.join(classpath) natives = os.path.join('..', self.dirnatives) @@ -1207,20 +1274,20 @@ with open(self.csvmethods, 'rb') as fh: methodsreader = csv.DictReader(fh) for row in methodsreader: - if int(row['side']) == side: + if int(row['side']) == side or int(row['side']) == 2: if row['name'] != row['searge']: names['methods'][row['searge']] = row['name'] with open(self.csvfields, 'rb') as fh: fieldsreader = csv.DictReader(fh) for row in fieldsreader: - if int(row['side']) == side: + if int(row['side']) == side or int(row['side']) == 2: if row['name'] != row['searge']: names['fields'][row['searge']] = row['name'] if self.has_param_csv: with open(self.csvparams, 'rb') as fh: paramsreader = csv.DictReader(fh) for row in paramsreader: - if int(row['side']) == side: + if int(row['side']) == side or int(row['side']) == 2: names['params'][row['param']] = row['name'] regexps = { @@ -1354,13 +1421,13 @@ methods = {} for row in methodsreader: #HINT: Only include methods that have a non-empty description - if int(row['side']) == side and row['desc']: + if (int(row['side']) == side or int(row['side']) == 2) and row['desc']: methods[row['searge']] = row['desc'].replace('*/', '* /') fields = {} for row in fieldsreader: #HINT: Only include fields that have a non-empty description - if int(row['side']) == side and row['desc']: + if (int(row['side']) == side or int(row['side']) == 2) and row['desc']: fields[row['searge']] = row['desc'].replace('*/', '* /') regexps = { @@ -1437,7 +1504,7 @@ self.runcmd(forkcmd) return True - def gathermd5s(self, side, reobf=False): + def gathermd5s(self, side, reobf=False, skip_fml=False): if not reobf: md5lk = {CLIENT: self.md5client, SERVER: self.md5server} else: @@ -1452,6 +1519,9 @@ class_path = '' else: class_path += '/' + if skip_fml: + if class_path.startswith('cpw/'): + continue for class_file in fnmatch.filter(filelist, '*.class'): class_name = class_path + os.path.splitext(class_file)[0] bin_file = os.path.normpath(os.path.join(path, class_file)) @@ -1548,6 +1618,8 @@ if not os.path.exists(outpathlk[side]): os.makedirs(outpathlk[side]) + reserved = ['CON', 'PRN', 'AUX', 'NUL', 'COM1', 'COM2', 'COM3', 'COM4', 'COM5', 'COM6', 'COM7', 'COM8', 'COM9', 'LPT1', 'LPT2', 'LPT3', 'LPT4', 'LPT5', 'LPT6', 'LPT7', 'LPT8', 'LPT9'] + # HINT: We extract the modified class files with closing(zipfile.ZipFile(jarlk[side])) as zipjar: for in_class in trgclasses: @@ -1561,6 +1633,23 @@ out_class = out_class.replace(self.nullpkg, '') if out_class[0] == '/': out_class = out_class[1:] + + rename = False + for res in reserved: + if out_class.upper().startswith(res): + rename = True + break + + if rename: + try: + f = open(os.path.join(outpathlk[side], '_' + out_class), 'wb') + f.write(zipjar.read(out_class)) + f.close() + self.logger.info('> Outputted %s to %s as %s', in_class.ljust(35), outpathlk[side], '_' + out_class) + except IOError: + self.logger.error('* File %s failed extracting for %s', out_class, in_class) + continue + try: zipjar.extract(out_class, outpathlk[side]) self.logger.info('> Outputted %s to %s as %s', in_class.ljust(35), outpathlk[side], out_class) @@ -1607,6 +1696,9 @@ sys.exit(1) for entry in newfiles: + if 'commands.py' in entry[0]: #FML, Disable updating of Commands.py + print 'Update to runtime/commands.py found, but disbled due to using fml' + continue if entry[3] == 'U': self.logger.info('Retrieving file from server : %s', entry[0]) cur_file = os.path.normpath(entry[0]) @@ -1627,6 +1719,9 @@ md5reoblk = {CLIENT: self.md5reobfclient, SERVER: self.md5reobfserver} outpathlk = {CLIENT: self.srcmodclient, SERVER: self.srcmodserver} src = {CLIENT: self.srcclient, SERVER: self.srcserver} + + if side == SERVER: #Noop out server side stuff + return # HINT: We need a table for the old md5 and the new ones md5table = {}