[Bug 1055730] Review Request: vdr-iptv - multicast IPTV transport stream plugin for VDR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1055730



--- Comment #1 from Mohamed El Morabity <pikachu.2014@xxxxxxxxx> ---
Some comments:

- same issue than in vdr-tvguide review: since the package is written in C++,
and since the CXXFLAGS variable is handled by the plugin Makefile, you must
also set CXXFLAGS when calling make, to keep consistency on build flags:
    make CFLAGS="%{optflags}" CXXFLAGS="%{optflags}" ...
  You should even add -fPIC to both CFLAGS and CXXFLAGS to handle build issues
on x86_64 platforms.

- the scripts deployed in %{vdr_configdir}/plugins/iptv/ can be called from
channels.conf when an IPTV channel is selected from VDR. Most of them are demo
scripts; they must be modified/copied to fit users' needs. I wonder if it's
worth marking them as %config(noreplace) so these scripts wouldn't be
overwritten by further updates if they were modified. Notice it's not a must,
only a suggestion.

- among those scripts, vlc2iptv contains a bad path to channels.conf. This
script should probably be modified to refer to the right path to channels.conf,
as defined by Fedora:

%prep
%setup -q -n %{vdr_plugin}-%{version}

# Fix paths in plugin scripts
sed -i "s|^CHANNELS_CONF=.*|CHANNELS_CONF=%{vdr_configdir}/channels.conf|; \
       
s|^CHANNEL_SETTINGS_DIR=.*/iptv|CHANNEL_SETTINGS_DIR=%{vdr_configdir}/plugins/%{vdr_plugin}|"
\
        iptv/vlc2iptv

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]