Bug 1849568 - Review Request: sourcetrail - a free and open-source interactive source explorer
Summary: Review Request: sourcetrail - a free and open-source interactive source explorer
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Tom Stellard
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-06-22 09:19 UTC by serge_sans_paille
Modified: 2020-07-23 06:25 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-07-23 06:25:48 UTC
Type: ---
Embargoed:
tstellar: fedora-review+


Attachments (Terms of Use)

Description serge_sans_paille 2020-06-22 09:19:49 UTC
Spec URL: https://sergesanspaille.fedorapeople.org/sourcetrail.spec
SRPM URL: https://sergesanspaille.fedorapeople.org/sourcetrail-2020.1.117-1.fc33.src.rpm
Description: Sourcetrail is a free and open-source cross-platform source explorer that helps you get productive on unfamiliar source code.
Fedora Account System Username: sergesanspaille

Comment 1 serge_sans_paille 2020-06-22 09:20:33 UTC
This is my first time packaging a graphical application, Can't say If I missed something on the xdg-desktop aspects...

Comment 2 Konrad Kleine 2020-06-22 10:47:57 UTC
As mentioned on IRC, when installing this on Fedora 32 I get this error:

$ dnf install https://sergesanspaille.fedorapeople.org/sourcetrail-2020.1.117-1.fc33.x86_64.rpm 
Last metadata expiration check: 0:11:59 ago on Mon 22 Jun 2020 10:37:00 AM CEST.
sourcetrail-2020.1.117-1.fc33.x86_64.rpm                                                2.0 MB/s | 3.3 MB     00:01    
Error: 
 Problem: conflicting requests
  - nothing provides libboost_filesystem.so.1.73.0()(64bit) needed by sourcetrail-2020.1.117-1.fc33.x86_64
  - nothing provides libboost_program_options.so.1.73.0()(64bit) needed by sourcetrail-2020.1.117-1.fc33.x86_64
(try to add '--skip-broken' to skip uninstallable packages)

Comment 3 Tom Stellard 2020-06-22 14:56:12 UTC
Since this project uses CMake, you should try to align the spec file with this proposal: https://fedoraproject.org/wiki/Changes/CMake_to_do_out-of-source_builds .  I don't know if all the new macros exist, but you could at least adopt the convention of not cd'ing into the build directory.  And also use _vpath_builddir macro for the build directory.

Comment 4 Tom Stellard 2020-06-25 15:05:09 UTC
Is there a reason why debuginfo is disabled?

Comment 5 serge_sans_paille 2020-06-26 18:28:51 UTC
@KonradKeline: that's because this package is buit for rawhide, which provides a higher version of boost. If this gets built for f32, the dependencies should be satisfied.
@Tom: no reason for debuginfo, and I updated the cmake lines.

Comment 6 serge_sans_paille 2020-07-02 12:07:08 UTC
There has been a release upsteam, that adds support for llvm-10. I've updated the spec etc accordingly.

Spec URL: https://sergesanspaille.fedorapeople.org/sourcetrail.spec
SRPM URL: https://sergesanspaille.fedorapeople.org/sourcetrail-2020.2.43-1.fc33.src.rpm

Comment 8 Tom Stellard 2020-07-09 14:14:14 UTC
You should add a comment explaining why parallel builds are disabled.  Otherwise, LGTM.

Comment 9 serge_sans_paille 2020-07-15 07:11:10 UTC
Parallel builds are now ok, I updated the uploaded files to reflect that.

Comment 10 Gwyn Ciesla 2020-07-20 15:00:40 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/sourcetrail

Comment 11 serge_sans_paille 2020-07-23 06:25:48 UTC
Uploaded as sourcetrail-2020.2.43-1.fc33


Note You need to log in before you can comment on or make changes to this bug.