Bug 1371502 - Review Request: Siril - Astronomical image (pre-)processing program
Summary: Review Request: Siril - Astronomical image (pre-)processing program
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-08-30 11:08 UTC by cyril
Modified: 2016-11-29 13:19 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-10-18 11:30:09 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description cyril 2016-08-30 11:08:05 UTC
Spec URL: https://free-astro.org/download/fedora/SPECS/siril.spec
SRPM URL: https://free-astro.org/download/fedora/SRPMS/siril-0.9.4-1.fc24.src.rpm
Description: Siril is an image processing tool specially tailored for noise reduction and improving the signal/noise ratio of an image from multiple captures, as required in astronomy.

It depends on ffms2 library which is not in fedora repositories but can be found here: http://koji.russianfedora.pro/koji/buildinfo?buildID=3691

An old and unmaintained version of siril is always here: https://www.rpmfind.net/linux/RPM/fedora/23/i386/s/siril-0.8-20.fc23.i686.html. This version should be removed or at least updated to the new one.

Comment 1 Igor Gnatenko 2016-08-30 11:12:42 UTC
Unfortunately ffms2 can't be put into fedora as it requires ffmpeg. Hence I'm closing this review request.

Feel free to submit this package to Russian Fedora or RPM Fusion! I would be happy to help.

Comment 2 cyril 2016-08-30 11:21:48 UTC
Where can I submit to RPM Fusion ?

Comment 3 Christian Dersch 2016-10-06 22:53:09 UTC
Is ffms2 really required? We have this old version 0.8 in offifial repo, would be nice if it could be updated… But as Igor mentioned: ffmpeg would be a blocker

Comment 4 Christian Dersch 2016-10-06 23:04:03 UTC
Seems to be possible to build without ffms2 => I'll investigate for an update

Comment 5 cyril 2016-10-07 21:01:13 UTC
(In reply to Christian Dersch from comment #4)
> Seems to be possible to build without ffms2 => I'll investigate for an update

Too late, I'll ask to https://bugzilla.rpmfusion.org/show_bug.cgi?id=4224

I'm one of the developer, and YES ffms2 is needed even if we can compile without it. Moreover, next release will use ffmpeg libraries for some features.

So the best thing to do now, is to orphan this old, very old 0.8 version.

Comment 6 Christian Dersch 2016-10-07 21:13:06 UTC
Well, according to http://free-astro.org/index.php/Siril:install#Dependencies ffms2 is optional? 

I can do the rework and update for Fedora, but only without problematic stuff. In this way it can pe part of Fedora Astronomy Spin, otherwise I have to remove it from there… Adding the nonfree extensions as some kind of plugin as rpmfusion package is not possible?

Comment 7 cyril 2016-10-07 21:23:43 UTC
(In reply to Christian Dersch from comment #6)
> Well, according to
> http://free-astro.org/index.php/Siril:install#Dependencies ffms2 is
> optional? 
> 
> I can do the rework and update for Fedora, but only without problematic
> stuff. In this way it can pe part of Fedora Astronomy Spin, otherwise I have
> to remove it from there… 

Sure, it is optional for compilation. When compiling, user can choose which part he wants in order to optimize the software. But for packages I want the whole piece.

> Adding the nonfree extensions as some kind of
> plugin as rpmfusion package is not possible?

I don't think it's possible without changing to much code.

I really understand that ffms2 is a problematic stuff, but as I said, it is getting worst in next release with new dependencies with ffmpeg libraries. And without ffms2, user can't open planetary avi file to work with.

So, I think that the best way is to try to review the package in rpm fusion and to remove the 0.8 version from Fedora. Indeed, this version is not really usable and should not be downloadable in modern distribution.

Comment 8 cyril 2016-10-07 21:43:33 UTC
I'm sorry for that by the way, but i did not know that before asking for the package review.

Comment 9 Christian Dersch 2016-10-07 23:03:49 UTC
(In reply to cyril from comment #7)
> 
> So, I think that the best way is to try to review the package in rpm fusion
> and to remove the 0.8 version from Fedora. Indeed, this version is not
> really usable and should not be downloadable in modern distribution.

Well, I think there should be an up to date version in official Fedora, even if it misses some features. I'm really happy to see that siril is alive and IMHO it is a great software for astronomical image processing, especially for amateur astronomers. Thats why I want to have it in Fedora Astronomy as an official feature.

IMHO the best solution would be siril in Fedora and siril-freeworld in rpmfusion providing the problematic stuff too. We are not the first packagers in that situation, confirm https://pkgs.rpmfusion.org/cgit/free/audacity-freeworld.git/tree/ vs 

You can also add an advice at your homepage then that users should get that package instead of Fedoras siril to get all functions. I can also add a hint to package description in our package that the Fedora version misses some features due to Fedora guidelines (to make clear it is not your fault). Win-Win situation IMHO, you can provide a full featured version via repositories and we have a free version in Fedora to provide a nice Fedora Astronomy experience (and siril there would be also an advertisement for your project).

Comment 10 cyril 2016-10-08 11:49:50 UTC
Hello Christian.

You convinced me. Let's do it.
However, current code does not permit to add a plugin in RPM Fusion. We will think about that on next release (0.9.5).

So could you do rework and update for Fedora the current version (0.9.4) ?

Comment 11 Christian Dersch 2016-10-09 12:13:56 UTC
Yes, I'll take care of that, nice!

Comment 12 cyril 2016-10-11 09:21:57 UTC
(In reply to Christian Dersch from comment #11)
> Yes, I'll take care of that, nice!

Thank you so much. What is the url where I can check the status ?

Comment 13 Christian Dersch 2016-10-14 12:30:17 UTC
Done, as you can see here I mostly used your spec to rework the package :) http://pkgs.fedoraproject.org/cgit/rpms/siril.git/tree/siril.spec

Builds are running, I'll submit updates for Fedora 24 and 25 to get 0.9.4 there :)

Comment 15 Fedora Update System 2016-10-14 21:23:34 UTC
siril-0.9.4-2.fc24 has been pushed to the Fedora 24 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-e4a425ce77

Comment 16 Fedora Update System 2016-10-14 21:54:13 UTC
siril-0.9.4-2.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-99c6b31855

Comment 17 Fedora Update System 2016-10-18 11:30:09 UTC
siril-0.9.4-2.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.

Comment 18 Fedora Update System 2016-10-24 02:48:44 UTC
siril-0.9.4-2.fc24 has been pushed to the Fedora 24 stable repository. If problems still persist, please make note of it in this bug report.

Comment 19 cyril 2016-11-29 09:10:30 UTC
Hello Christian.

There is a new upstream version: https://free-astro.org/download/siril-0.9.5.tar.bz2
Dependency to libgif has been removed but new dependencies to libffmpeg has been added. But do not worry, these dependencies are not mandatory.

Should I open a new issue when update is necessary ?

Best regards,

Comment 20 Christian Dersch 2016-11-29 13:19:12 UTC
Yes, please open new issues for new updates, thanks :) 0.9.5 will be imported today.


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