Fixed #148 - input dir path doesn't need to end with '/'

Author: chinyeungli

about_code_tool/about.py

@@ -1332,12 +1332,12 @@ def generate_attribution(self, template_path=None, limit_to=None):
         #if not_process_components:
         for component in not_process_components:
             for about_object in self:
-                # FIXME: what is the meaning of this partition?
-                # PO created the var some_path to provide some clarity
-                # but what does the second element means?
-                file_name = about_object.location.partition(self.location)[2]
-                # FIXME: a path starting with / is NOT relative
-                about_relative_path = '/' + file_name
+                # The about_object.location is the absolute path of the ABOUT
+                # file. The purpose of the following partition is to match
+                # the about_file's location with the input list.
+                about_relative_path = about_object.location.partition(
+                                                normpath(self.location))[2]
+
                 """if limit_to:
                     try:
                         not_process_components.remove(about_relative_path)

about_code_tool/tests/test_about.py

@@ -606,7 +606,7 @@ def test_remove_blank_lines_and_no_colon_fields(self):
             self.assertEqual(expected_warnings[i][0], w.code)
             self.assertEqual(expected_warnings[i][1], w.field_value)
 
-    # FIXME: This is feeling because there is no component list provided
+    # FIXME: This is failing because there is no component list provided
     def FAILING_test_generate_attribution_with_custom_template(self):
         expected = (u'notice_text:'
                     'version:2.4.3'
@@ -618,7 +618,7 @@ def FAILING_test_generate_attribution_with_custom_template(self):
         result = collector.generate_attribution(template)
         self.assertEqual(expected, result)
 
-    # FIXME: This is feeling because there is no component list provided
+    # FIXME: This is failing because there is no component list provided
     def FAILING_test_generate_attribution_with_default_template(self):
         f = open(join(TESTDATA_DIR, 'attrib/attrib.html'))
         expected = f.read()
@@ -685,7 +685,5 @@ def test_get_custom_field_keys(self):
                     'redistribute_sources', 'about_format', 'usage',
                     'scm_path', 'scm_tool', 'scm_rev', 'scm_tag',
                     'organization']
-        print(result)
         self.assertEqual(result, expected)
 
-