We are using mercurial in our environment and for some reason we are seeing a number of __init__.py (blank files for module inclusion in python) are being added to our diff file. Here is the process we used to reproduce the issue:
1. Clone a branch in the repo. 2. create a new file with some text in it (i.e. mytestfile.py). 3. Add a new file in the repo with "hg add mytestfile.py" 4. hg commit -m "This is a test" ./mytestfile.py 5. rbt post Here is what the diff of the new file looks like with a simple "hg diff" before running commit: diff --git a/python/tools/mytestfile.py b/python/tools/mytestfile.py new file mode 100644 --- /dev/null +++ b/python/mytestfile.py @@ -0,0 +1,1 @@ +print "This is a test" Attached please find the diff that was downloaded from Review Board. What you'll find is that the diff for the new file that was added does make it up there but for some reason ~50+ __init__.py files are also removed and then added to the diff. It causes the review request to look very messy. The diff file I added was downloaded directly from review board and then I snipped the end of it so the file wouldn't be too large. -- Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/ Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/ Happy user? Let us know! https://www.reviewboard.org/users/ --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
diff --git a/python/tools/mytestfile.py b/python/tools/mytestfile.py new file mode 100644 --- /dev/null +++ b/python/tools/mytestfile.py @@ -0,0 +1,1 @@ +print "This is a test" diff -r 74840703b2fc -r 91ae1954c259 ../../nmc_ui/test/management/commands/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../nmc_ui/test/management/commands/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../gui/apps/config/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../gui/apps/config/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../nmc_ui/apps/config/accounts/management/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../nmc_ui/apps/config/accounts/management/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../nasuni_test/ui/accounts/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../nasuni_test/ui/accounts/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../gui/apps/nfs/templatetags/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../gui/apps/nfs/templatetags/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../gui/apps/certs/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../gui/apps/certs/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../nasuni_test/config/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../nasuni_test/config/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../tests/ui/website/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../tests/ui/website/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../tests/feature/snmp/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../tests/feature/snmp/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../gui/apps/events/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../gui/apps/events/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../nasuni/gui_common/accountsbase/templatetags/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../nasuni/gui_common/accountsbase/templatetags/__init__.py Mon Dec 19 16:57:17 2016 -0500 diff -r 74840703b2fc -r 91ae1954c259 ../../gui/apps/ftp/migrations/__init__.py --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/../../gui/apps/ftp/migrations/__init__.py Mon Dec 19 16:57:17 2016 -0500 …This goes on for a while finding all the __init__.py files in our repository but don’t want to list them all here.
