Welcome, Guest. Please login or register.

Pages: [1]
Print
Author Topic: General patches thread  (Read 6616 times)
Vaporice
Community member

Posts: 13


View Profile Email
« on: October 19, 2009, 09:29:21 PM »

As there's no thread for this yet, barra asked me to make this thread.
The thread is meant for those little fixes that don't really deserve a separate thread but need to be reviewed.
Just post the trac ticket here =)

my fix (a little bug in the outline renderer):
http://parpg-trac.cvsdude.com/parpg/ticket/123
A minor thing (that's what the threads for  Grin) but it was a good practice to get to know the parpg source better

(for those that don't know me, i'm making a introduction post now)
« Last Edit: October 19, 2009, 11:11:16 PM by Vaporice » Logged
mvBarracuda
Admin
Community member

Posts: 1308



View Profile Email
« Reply #1 on: October 19, 2009, 10:43:17 PM »

Thanks for the patch vaporice :-) And welcome on the forums, nice to have you around. As soon as some programmer finds the time, the patch will be reviewed and added to SVN after that if no issues show up. Enjoy the stay!
Logged
maximinus
Community member

Posts: 694



View Profile Email
« Reply #2 on: January 29, 2010, 03:09:52 PM »

No activity on this thread since October 20th. I understand the intention of the thread but it's clearly not being used, so I'm un-stickying it (is that a verb?).

If you disagree, post below..
Logged

Science is open-source religion
shevegen
Community member

Posts: 705



View Profile
« Reply #3 on: January 29, 2010, 03:37:01 PM »

Noone wants to send patches!!!
Logged

Cleaning away the bureaucracy in PARPG to make our life easier.
przemo_li
Community member

Posts: 15


View Profile Email
« Reply #4 on: March 16, 2010, 08:48:00 AM »

How to send useful path ??

here is what i got with "svn diff"

Quote
[przemo@null scripts]$ cat diff_path
Index: dialogue.py
===================================================================
--- dialogue.py (revision 527)
+++ dialogue.py (working copy)
@@ -130,7 +130,6 @@
                 if end_cb:
                     end_cb()
                 logging.debug("Reached the end")
-                return
 
     def continueWithResponse(self, section_name, response):
         """Reply to a response in a section and continue executing dialogue

removed unnecessary return.
Logged
b0rland
Community member

Posts: 105



View Profile Email
« Reply #5 on: March 16, 2010, 09:26:16 AM »

That patch does work if you cut it after "=========="
It'd be more useful with paths relative to project root: you can do that by running svn diff in project root instead of under 'scripts' directory.

For more then one file, however, patches generated by svn diff aren't very good: patch program complains and you need to cut parts of them to prevent that. Typical way of generating patches is having a separate 'clean' checkout in a different directory and run:
Code:
diff -urN parpg  parpg_clean > my_fix.patch

You will probably want to review that file afterwards and make sure it only has the changes you wanted to submit.
Logged
przemo_li
Community member

Posts: 15


View Profile Email
« Reply #6 on: March 16, 2010, 09:32:10 AM »

so you mean

cd parpg_clean && svn up && cd ../parpg && svn up && cd .. && diff -urN parpg  parpg_clean > my_fix.patch && vi my_fix.patch

??
Logged
b0rland
Community member

Posts: 105



View Profile Email
« Reply #7 on: March 16, 2010, 10:11:14 AM »

right. Sorry, forgot to mention what 'parpg' and 'parpg_clean' are Smiley.  As might or might not be obvious from the names, parpg_clean is supposed to be clean and fresh svn checkout and parpg is your working copy with your changes and also updated to a fresh svn version.
Logged
b0rland
Community member

Posts: 105



View Profile Email
« Reply #8 on: March 16, 2010, 10:41:21 AM »

Re this specific patch, why do you thing 'return' isn't needed? Note that it's under while True, so unless this return is called, it will iterate over and over, even if EndException is triggered.
Logged
przemo_li
Community member

Posts: 15


View Profile Email
« Reply #9 on: March 18, 2010, 11:21:12 AM »

I got it. Sorry.


used my command and got too big diff patch Sad :
Code:
[przemo@null Development]$ wc -l my_fix.patch
424749 my_fix.patch
« Last Edit: March 18, 2010, 12:05:58 PM by przemo_li » Logged
mvBarracuda
Admin
Community member

Posts: 1308



View Profile Email
« Reply #10 on: March 19, 2010, 02:23:04 PM »

The whole patch submission process is described in detail at the wiki. Check out this articles:
http://wiki.parpg.net/How_to_Submit_a_Patch
http://wiki.parpg.net/Patches
Logged
przemo_li
Community member

Posts: 15


View Profile Email
« Reply #11 on: March 19, 2010, 09:43:21 PM »

Hmm, I've found that UT for console requires whole game loaded. So far someone just initialized console class with None wich cause error and ut fails. IMHO the best is to comment out all tests. So they want be counted in tests, but code will wait for better times. Hand test should be enough.

So here is patch that comment out whole tests/test_console.py
 revision is: "Updated to revision 535."

Code:
[przemo@null parpg]$ svn diff
Index: tests/test_console.py
===================================================================
--- tests/test_console.py       (revision 535)
+++ tests/test_console.py       (working copy)
@@ -16,33 +16,35 @@
 #   along with PARPG.  If not, see <http://www.gnu.org/licenses/>.
 
 
-import unittest
-from scripts.console import Console
+#import unittest
+#from scripts.console import Console
 
-class test_console(unittest.TestCase):
-    def setUp(self):
-        self.con=Console(None)
-        self.invalString="Invalid command, enter help for more information"
-        pass
-    
-    def tearDown(self):
-        pass
-
-    def testConsoleCommandHelp(self):
-        """ Test the help console command """
-        
-        self.assertNotEqual(self.con.handleHelp("help"),self.invalString)
-        self.assertNotEqual(self.con.handleConsoleCommand("help"),
-                            self.invalString)
-
-    def testConsoleCommandPython(self):
-        """ Test the python console command """
-        self.assertEqual(self.con.handlePython("python 1+1"),"2")
-        self.assertEqual(self.con.handleConsoleCommand("python 1+1"),"2")
-      
-    def testInvalid(self):
-        """Test an invalid console command """
-
-        self.assertEqual(self.con.handleConsoleCommand("invalid"),
-                         self.invalString)
-
+#class test_console(unittest.TestCase):
+#    def setUp(self):
+# FIXME        # Below wont work as it would require running whole game
+# all test must be done ingame till any better idea
+#      self.con=Console(None)
+#        self.invalString="Invalid command, enter help for more information"
+#        pass
+#    
+#    def tearDown(self):
+#        pass
+#
+#    def testConsoleCommandHelp(self):
+#        """ Test the help console command """
+#        
+#        self.assertNotEqual(self.con.handleHelp("help"),self.invalString)
+#        self.assertNotEqual(self.con.handleConsoleCommand("help"),
+#                            self.invalString)
+#
+#    def testConsoleCommandPython(self):
+#        """ Test the python console command """
+#        self.assertEqual(self.con.handlePython("python 1+1"),"2")
+#        self.assertEqual(self.con.handleConsoleCommand("python 1+1"),"2")
+#      
+#    def testInvalid(self):
+#        """Test an invalid console command """
+#
+#        self.assertEqual(self.con.handleConsoleCommand("invalid"),
+#                         self.invalString)
+#
[przemo@null parpg]$ svn diff >> no_console_tests.patch

PS why tests/test_* ? "from tests import test_console" looks ugly :|, and test are auto detected so name makes no difference.
« Last Edit: March 19, 2010, 09:45:37 PM by przemo_li » Logged
Pages: [1]
Print
Jump to: