Bug 1766874 - Review Request: jaero - A SatCom ACARS demodulator and decoder for the Aero standard
Summary: Review Request: jaero - A SatCom ACARS demodulator and decoder for the Aero s...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Vitaly
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-10-30 06:49 UTC by Vasiliy Glazov
Modified: 2019-10-31 14:09 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-10-31 14:09:26 UTC
Type: Bug
Embargoed:
vitaly: fedora-review+


Attachments (Terms of Use)

Description Vasiliy Glazov 2019-10-30 06:49:30 UTC
Spec URL: https://raw.githubusercontent.com/RussianFedora/jaero/master/jaero.spec
SRPM URL: https://kojipkgs.fedoraproject.org//work/tasks/4983/38654983/jaero-1.0.4.11-1.fc32.src.rpm

Description:
JAERO is a program that demodulates and decodes Classic Aero ACARS (Aircraft
Communications Addressing and Reporting System) messages sent from satellites to
aeroplanes (SatCom ACARS), commonly used when planes are beyond VHF range.

Demodulation is performed using the soundcard.

Such signals are typically around 1.5Ghz and can be received with a
low-gain antenna that can be home-brewed in conjunction with an
RTL-SDR dongle.

Comment 2 Vitaly 2019-10-30 14:22:39 UTC
> Requires:       unzip

1. Add %{?_isa}.

> BuildRequires:  kiss-fft-static

2. kiss-fft-devel

> sed -i '/kiss_fft130\/kiss_fft/d' JAERO/JAERO.pro

3. Please replace multiple sed calls with a single one.

> cd JAERO/build

4. I think it will be better to use pushd JAERO/build ... popd instead.

5, Please add AppData metafile and send it to upstream.

6. Add comments to used in project patches.

Comment 3 Vasiliy Glazov 2019-10-31 07:56:12 UTC
Spec URL: https://raw.githubusercontent.com/RussianFedora/jaero/master/jaero.spec
SRPM URL: https://kojipkgs.fedoraproject.org//work/tasks/8574/38678574/jaero-1.0.4.11-2.fc32.src.rpm

1. Added %{?_isa} to unzip.

2. Gudelines say that need BR: ...-static https://docs.fedoraproject.org/en-US/packaging-guidelines/#_packaging_static_libraries
And because -devel provides static I set only one of them.

3. Multiple sed replaced.

4. Used pushd/popd.

5. I don't have AppData.

6. Comments to patches added.

Comment 4 Vitaly 2019-10-31 10:43:48 UTC
The license is MIT and LGPLv2.1:

./JAERO/gui_classes/console.cpp: LGPL (v2.1 or v3)
./JAERO/gui_classes/console.h: LGPL (v2.1 or v3)

> mkdir JAERO/build

Can be moved to %prep.

> I don't have AppData.

You should add it manually and send pull request to upstream.

Comment 6 Vitaly 2019-10-31 11:05:52 UTC
LGTM now. Package approved.

Comment 7 Gwyn Ciesla 2019-10-31 14:07:04 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/jaero


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