Bug 1723013 - Review Request: mne-cpp - A Framework for Electrophysiology [NEEDINFO]
Summary: Review Request: mne-cpp - A Framework for Electrophysiology
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2019-06-22 02:16 UTC by b38ghk29
Modified: 2020-09-25 07:36 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed:
Type: ---
b38ghk29: needinfo? (nobody)

Attachments (Terms of Use)

Description b38ghk29 2019-06-22 02:16:12 UTC
Spec URL: https://drive.google.com/file/d/1SxTNt7ZTMgHkEihgUvZhtjR0vny5XQPo/view?usp=sharing
SRPM URL: https://drive.google.com/file/d/1e3Y5CKKvGXYrKF6qw-WsudN2al0MRcQE/view?usp=sharing
Description: The mne-cpp (cpp = c++) code base is a framework for acquiring/capturing electrophysiology data (in real-time) and/or browsing and analyzing such data.  It is a re-write of the original C code base.  See this URL for some of the different programs available in the mne-cpp distribution, https://www.mne-cpp.org/index.php/info/

Fedora Account System Username:  redhat bugzilla email b38ghk29@contbay.com

I have done build and install so far only on fc28 updated to have Qt 5.11.3.

"sudo dnf builddep mne-cpp.spec" installed dependent Qt packages for build.

"mock -v mne-cpp-master-1.0.fc28.src.rpm" built the source RPM and existed with status 0.

Comment 1 David Auer 2019-06-22 14:13:52 UTC
These are not direct download links, did you read https://fedoraproject.org/wiki/Package_Review_Process ?

This way the fedora-review tool works and people need less clicks:
Spec URL: https://drive.google.com/uc?export=download&id=1SxTNt7ZTMgHkEihgUvZhtjR0vny5XQPo
SRPM URL: https://drive.google.com/uc?export=download&id=1e3Y5CKKvGXYrKF6qw-WsudN2al0MRcQE

I am also a bit confused about your Fedora account system username and the fact you are using Fedora 28 (it is end of life).

Comment 2 David Auer 2019-06-22 14:41:57 UTC
Okay, it turns out that fedora-review looks for the file ending, so maybe this hack works:

Spec URL: https://drive.google.com/uc?export=download&id=1SxTNt7ZTMgHkEihgUvZhtjR0vny5XQPo#.spec
SRPM URL: https://drive.google.com/uc?export=download&id=1e3Y5CKKvGXYrKF6qw-WsudN2al0MRcQE#.src.rpm

Comment 3 b38ghk29 2019-07-02 02:00:56 UTC
I have rebuilt/created RPM's on fc29 and have direct download URL's, but where do you edit the previous URL entries?

Comment 4 David Auer 2019-07-02 10:28:08 UTC
AFAIK you can not edit previous messages. Just post the URLs in a new comment. Unfortunately my tries to convert your URLs to direct download weren't successful. 

When I last commented I took a short peek at your spec file and if I recall correctly there are some things that need to be fixed, if you haven't already please make sure your Version/Release, License and makro usage meets the guidelines (https://docs.fedoraproject.org/en-US/packaging-guidelines/). I will try to make a full unofficial review in the next days. (I am not a packager yet so all of this is unofficial!) 

I think downloading sources via curl in %setup is not allowed, but not entirely sure in this case, because it wouldn't happen for Fedora but only for rhel builds. 

Additionally it would be a good idea to do a koji sratch build (also post a link when it's done) and I personally ran the fedora-review tool on my own package, it might just help you to find some mistakes and fix them yourself.

Comment 5 b38ghk29 2019-07-02 18:14:07 UTC
Thank you for the info and the update.  I saw in the packaging docs that downloading is discouraged, but given there is no release yet of the code (or official package), then I was not sure what else to do besides fetch the source from github.  I did run mock, but I will look at the other things you mentioned.  For now here are the updated URLs,

Spec URL: https://drive.google.com/uc?export=download&id=1SxTNt7ZTMgHkEihgUvZhtjR0vny5XQPo
SRPM URL: https://drive.google.com/uc?export=download&id=1Os82KO4Bmqv5uARWZa-PUm3cXOsrH1sV

Comment 6 Artur Frenszek-Iwicki 2019-07-06 10:38:14 UTC
>Version:        master
>Release:        1.0%{?dist}
You should use the upstream version number (1.0.0) and include snapshot info in the Release field. See the Versioning Guidelines:

>Source0:        https://github.com/mne-tools/mne-cpp/archive/master.zip
Related to the point above - you should explicitly name a specific commit here. "master.zip" is a moving target.

>curl -s https://raw.githubusercontent.com/jcfr/qt-easy-build/5.10.0/Build-qt.sh -o Build-qt.sh
This will fail since official package builds are done with disabled Internet access. Everything that's needed for building the package should be listed in Source: and Patch: tags.

Comment 7 Robert-André Mauchin 🐧 2019-07-13 14:14:21 UTC
 - This is a no: you don't have internet access in Koji/Mock during the build

%if 0%{?rhel}
echo in_setup_for_RedHat
curl -s https://raw.githubusercontent.com/jcfr/qt-easy-build/5.10.0/Build-qt.sh -o Build-qt.sh
chmod 755 Build-qt.sh
./Build-qt.sh -y -c -j4 -q $HOME/qt

Comment 8 Package Review 2020-07-13 00:45:28 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time. We're sorry
it is taking so long. If you're still interested in packaging this software
into Fedora repositories, please respond to this comment clearing the

You may want to update the specfile and the src.rpm to the latest version
available and to propose a review swap on Fedora devel mailing list to increase
chances to have your package reviewed. If this is your first package and you
need a sponsor, you may want to post some informal reviews. Read more at

Without any reply, this request will shortly be considered abandoned
and will be closed.
Thank you for your patience.

Comment 9 b38ghk29 2020-07-17 15:26:05 UTC
OK, so per your feedback above on 2019-07-13 14:14:21 UTC, then the current Qt distribution has to be built from source prior to building the software applications that use it?

Comment 10 Robert-André Mauchin 🐧 2020-07-18 15:41:16 UTC
# CentOS/RedHat needs Qt 5.10 to be downloaded and built (no rpm for install)
%if 0%{?rhel}
echo in_setup_for_RedHat
curl -s https://raw.githubusercontent.com/jcfr/qt-easy-build/5.10.0/Build-qt.sh -o Build-qt.sh
chmod 755 Build-qt.sh
./Build-qt.sh -y -c -j4 -q $HOME/qt

If Qt > 5.10 is required then you can't support RHEL < 7. You can target RHEL 8 (not sure of Qt's version here, please check).

 - Secondly you need to use nacros:

qmake --version
qmake -makefile -recursive



 - Do not install in /usr/local, use %{_prefix} macro which is /usr

 - No idea what you are trying to do here:

# echo %{buildroot}
# copy over subdirs from build
mkdir -p %{buildroot}/%{_prefix}/mne-cpp
tar cpf - ./bin ./lib ./doc | (cd %{buildroot}/%{_prefix}/mne-cpp; tar xpf -)

 - Also

Source0:        https://github.com/mne-tools/mne-cpp/archive/master.zip

you can't just pount to master like this, you need to d/l a specific release, if not are provided, then target a specific commit. It seems 0.1.4 is the latest release.

Comment 11 b38ghk29 2020-09-25 00:50:26 UTC
Please correct me if I am wrong, but my understanding is that a *src* rpm is still required, and so the source code must build from scratch (apart from subsequently creating a binary RPM).

Also please advise on what Fedora releases are acceptable to compile/test on, i.e., fc31 or fc32.  The reason I ask is because with newer compilers, the current 0.1.6 pre-release may not compile w/o warnings.

Comment 12 Artur Frenszek-Iwicki 2020-09-25 07:36:45 UTC
I don't think we have a requirement as to which release one has is allowed to use for building SRPMs for review, though I guess the unspoken expectation is "something that's not EOL". That being said, you can use mock, or koji scratch builds to verify that your package builds fine on Fedora Rawhide.

Once you have a successful scratch build, it's good to add a link to it alongside the SPEC and SRPM - just for showing reviewers that the package builds fine.

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