FIXME note for evil code

1 files changed, 3 insertions, 0 deletions

diff --git a/lib/Youri/Repository/Mandriva_upload.pm b/lib/Youri/Repository/Mandriva_upload.pm
index 381aee5..e983a4a 100644
--- a/lib/Youri/Repository/Mandriva_upload.pm
+++ b/lib/Youri/Repository/Mandriva_upload.pm
@@ -386,6 +386,9 @@ sub _get_section {
         }
     }
 
+    # FIXME: doing this here is wrong; this way the caller can never know if
+    # a section was actually found or not; should return undef and let the
+    # caller set a default (Note: IIRC PLF|Zarb has this right, see there) -spuk
     print STDERR "Warning: Can't guess destination: section missing, defaulting to contrib/release\n" unless $section;
     $section ||= 'contrib/release';