https://bugzilla.novell.com/show_bug.cgi?id=891829

https://bugzilla.novell.com/show_bug.cgi?id=891829#c0

Summary: source_validator incorrectly parses conditionals in preamble Classification: openSUSE Product: openSUSE.org Version: unspecified Platform: Other OS/Version: Other Status: NEW Severity: Major Priority: P5 - None Component: BuildService AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: sbrabec@suse.com QAContact: adrian@suse.com CC: ro@suse.com Found By: --- Blocker: ---

Created an attachment (id=602295) --> (http://bugzilla.novell.com/attachment.cgi?id=602295) util-linux.spec file passed to rpmbuild

When processing new util-linux package, I got following error:

error: Duplicate License entries in package: (main package) Aborting: service call failed: /usr/lib/obs/service/source_validator --outdir /tmp/tmpUOqaab

However rpmbuild -ba util-linux.spec does not see any problems of this spec file.

This statement is apparently incorrect.

Looking at the spec file, I see: One License tag inside "%if %build_util_linux" - this one should be active.

Second License tag inside "%if %build_python_libmount" and inside %else of "%if %build_util_linux" - this should be inactive.

Third License tag inside "%if %build_util_linux_systemd" and inside %else of "%if %build_util_linux" - this should be inactive.

According to the initial evaluation of variables, we should have: %define build_util_linux 1 %define build_util_linux_systemd 0 %define build_python_libmount 0 %define verify_sig 0

But if I look into preprocessed /tmp/check_if_valid_source_dir-5rRp5R/tmp.spec, I see that all are supposed to be 1.

Note that this bug is triggered by format_spec_file (see bug 891152), which forces placing License tags to each %if branch.