Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 1606076

Summary: rapidsvn: FTBFS in Fedora rawhide
Product: [Fedora] Fedora Reporter: Mohan Boddu <mboddu>
Component: rapidsvnAssignee: Tim Jackson <rpm>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: releng, rpm, swt
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-12-04 21:49:04 UTC Type: ---
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: 1423041, 1555378, 1602938    
Attachments:
Description Flags
build.log
none
root.log
none
state.log none

Description Mohan Boddu 2018-07-20 18:19:14 UTC
rapidsvn failed to build from source in Fedora rawhide

https://koji.fedoraproject.org/koji/taskinfo?taskID=28227635


For details on the mass rebuild see:

https://fedoraproject.org/wiki/Fedora_29_Mass_Rebuild
Please fix rapidsvn at your earliest convenience and set the bug's status to
ASSIGNED when you start fixing it. If the bug remains in NEW state for 8 weeks,
rapidsvn will be orphaned. Before branching of Fedora 30,
rapidsvn will be retired, if it still fails to build.

For more details on the FTBFS policy, please visit:
https://fedoraproject.org/wiki/Fails_to_build_from_source

Comment 1 Mohan Boddu 2018-07-20 18:19:22 UTC
Created attachment 1467273 [details]
build.log

file build.log too big, will only attach last 32768 bytes

Comment 2 Mohan Boddu 2018-07-20 18:19:27 UTC
Created attachment 1467274 [details]
root.log

file root.log too big, will only attach last 32768 bytes

Comment 3 Mohan Boddu 2018-07-20 18:19:30 UTC
Created attachment 1467275 [details]
state.log

Comment 4 Tim Jackson 2018-08-06 20:27:18 UTC
duplicate of #1424240

Comment 5 Tim Jackson 2018-08-06 20:45:33 UTC
I've tried building from the latest upstream git on F28; it fails, apparently less severely though still fails due to a bunch of type conversion errors. See https://github.com/RapidSVN/RapidSVN/issues/35

Comment 6 Tim Jackson 2018-08-06 20:50:41 UTC
Posted to upstream mailing list:
http://rapidsvn.tigris.org/ds/viewMessage.do?dsForumId=671&dsMessageId=3623661

Comment 7 Tim Jackson 2018-08-06 20:51:37 UTC
*** Bug 1424240 has been marked as a duplicate of this bug. ***

Comment 8 Jan Kurik 2018-08-14 11:03:28 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 29 development cycle.
Changing version to '29'.

Comment 9 Scott Talbert 2018-11-06 01:28:01 UTC
(In reply to Tim Jackson from comment #5)
> I've tried building from the latest upstream git on F28; it fails,
> apparently less severely though still fails due to a bunch of type
> conversion errors. See https://github.com/RapidSVN/RapidSVN/issues/35

Latest upstream git seems to build OK if you switch to use wxWidgets 3.0 (change wxGTK-devel to wxGTK3-devel).

Comment 10 Tim Jackson 2018-11-25 21:22:08 UTC
Many thanks for the tip Scott! The change to wxWidgets 3.0 does indeed solve a lot of issues. There are some other changes that are needed to the spec file, but it fundamentally builds now. I'll push an update soon.

Comment 11 Tim Jackson 2018-12-03 22:07:09 UTC
So I finally pushed a fix to git which cleans up the whole spec and builds with the latest upstream version... and it now fails on F-30 because some of the build tools use Python 2. So, down another rabbit hole. At least tools/msgfmt.py fails to run with the Python 3 interpreter. If anyone has the desire to help then it will be much appreciated, since upstream is not very active.

Comment 12 Tim Jackson 2018-12-03 22:09:23 UTC
*** Bug 1556310 has been marked as a duplicate of this bug. ***

Comment 13 Tim Jackson 2018-12-03 22:12:20 UTC
for i in de fr hu it_IT pt_BR ru es uk zh_CN ja ; do \
    python3 ../../../tools/msgfmt.py -o ./$i/rapidsvn.mo ./$i/rapidsvn.po ; \
done
  File "../../../tools/msgfmt.py", line 86
    0x950412deL,       # Magic
              ^
SyntaxError: invalid syntax

Comment 14 Tim Jackson 2018-12-03 22:19:40 UTC
so, number with an "L" suffix is a long integer, apparently: https://docs.python.org/2/library/stdtypes.html

Looks like it can just be removed in Python3 : https://stackoverflow.com/questions/24059842/l-suffix-in-long-integer-in-python-3-x

but then it's:

  File "../../../tools/msgfmt.py", line 113
    except IOError, msg:

:-(

Comment 15 Scott Talbert 2018-12-04 04:09:13 UTC
(In reply to Tim Jackson from comment #11)
> So I finally pushed a fix to git which cleans up the whole spec and builds
> with the latest upstream version... and it now fails on F-30 because some of
> the build tools use Python 2. So, down another rabbit hole. At least
> tools/msgfmt.py fails to run with the Python 3 interpreter. If anyone has
> the desire to help then it will be much appreciated, since upstream is not
> very active.

You can just BuildRequire: /usr/bin/python for now and still use Python 2.  See:
https://fedoraproject.org/wiki/Changes/Move_usr_bin_python_into_separate_package

Comment 16 Tim Jackson 2018-12-04 21:18:58 UTC
Thanks. Having looked further, this is some kind of standard tool which is used instead of gettext msgfmt, and it's actually just a makefile fix to use gettext instead (which I believe was the case in previous builds, but upstream is now using msgfmt.py as default instead)

Comment 17 Tim Jackson 2018-12-04 21:47:38 UTC
Now successfully built in rawhide!
https://koji.fedoraproject.org/koji/taskinfo?taskID=31269439

Thanks for the help Scott!