Bug 1215643

Summary: ERROR: Broken dependency: KASComp 1.8>KASComp 1.8
Product: [Fedora] Fedora Reporter: Raphael Groner <projects.rg>
Component: lazarusAssignee: Joost van der Sluis <joost>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: high Docs Contact:
Priority: unspecified    
Version: rawhideCC: joost
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-04-29 10:07:42 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1208911    
Attachments:
Description Flags
doublecmd.spec none

Description Raphael Groner 2015-04-27 11:31:09 UTC
Description of problem:
lazbuild produces FTBFS in F22 and rawhide. F20 with lazarus 1.0.4 works.

Version-Release number of selected component (if applicable):
lazarus-1.4-0.2.RC2.fc23
lazarus-1.4-0.2.RC2.fc22

How reproducible:
yes

Steps to Reproduce:
1. build doublecmd (spec file is attached)
2.
3.

Actual results:
ERROR: Broken dependency: KASComp 1.8>KASComp 1.8

Expected results:
no error

Additional info:
Why did you push a Release Candidate ("RC2") to F22?
Raising Severity cause it makes rpmbuild unusable.

Comment 1 Raphael Groner 2015-04-27 11:36:23 UTC
Created attachment 1019299 [details]
doublecmd.spec

Comment 2 Raphael Groner 2015-04-27 11:57:16 UTC
Sorry, confused the versions.

lazarus-1.0.14-1.fc20    - works?
lazarus-1.2-4.fc21       - works
lazarus-1.4-0.2.RC2.fc22 - fails
lazarus-1.4-0.2.RC2.fc23 - fails

Comment 3 Joost van der Sluis 2015-04-27 13:39:44 UTC
From the build log from your Review Request: 

Compiling dcdatetimeutils.pas
dcdatetimeutils.pas(216,28) Hint: Converting the operands to "Int64" before doing the multiply could prevent overflow errors.
dcdatetimeutils.pas(218,33) Hint: Converting the operands to "Int64" before doing the multiply could prevent overflow errors.
dcdatetimeutils.pas(475,20) Hint: Mixing signed expressions and longwords gives a 64bit result
dcdatetimeutils.pas(475,20) Hint: Converting the operands to "Int64" before doing the multiply could prevent overflow errors.
dcdatetimeutils.pas(475,42) Hint: Converting the operands to "Int64" before doing the multiply could prevent overflow errors.
Compiling dcxmlconfig.pas
Compiling dcprocessutf8.pas
dcprocessutf8.pas(24,5) Fatal: Can't find unit UTF8Process used by DCProcessUtf8
TExternalToolList.Run Exception: /builddir/build/BUILD/doublecmd-r6000/components/doublecmd/dcprocessutf8.pas(24,5) Fatal: Can't find unit UTF8Process used by DCProcessUtf8

So there's a bug in the code of doublecmdt. My guess is that the doublecmd-package lacks a dependency on the LazUtils package. Update the code of doublecmdt to make it work with Lazarus 1.4, or ask upstream.

But that your package has a bug in it, does not mean that you have to file a bug-report on one of the libraries it uses.

And F22 is currently in beta, Lazarus 1.4rc2 has been pushed into F22 during it's alpha status. This is what the alpha/beta stages are for: test if there are any problems with packages if they have to work together. And as Lazarus 1.4 is already released, 1.4 will make it in time for the F22 final-release.

Comment 4 Raphael Groner 2015-04-28 12:15:15 UTC
Thank you anyway for the quick response. I wrote an e-mail to upstream about the FTBFS of doublecmd.

Comment 5 Raphael Groner 2015-04-28 12:57:34 UTC
Does this work with official 1.4.0 release from upstream? Rawhide still has RC2.

Comment 6 Joost van der Sluis 2015-04-29 10:07:42 UTC
(In reply to Raphael Groner from comment #5)
> Does this work with official 1.4.0 release from upstream? Rawhide still has
> RC2.

No, of course not. There are only minor changes between rc2 and the final. There are no new units added or removed in between.

Comment 7 Raphael Groner 2015-05-01 12:49:57 UTC
FYI: Lazarus 1.4 has a bug in lazbuild utility. The developer of doublecmd wrote about it to Lazarus bugtracker and it was fixed (revisions 48892,48893). This fix will be included in Lazarus 1.4.2 .