[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #33 from Dominic Robinson  
---
Sadly still not liking it:

debug1: Offering RSA public key: /home/dcrdev/.ssh/id_rsa
debug3: send_pubkey_test
debug3: send packet: type 50
debug2: we sent a publickey packet, wait for reply
debug3: receive packet: type 51
debug1: Authentications that can continue: publickey
debug1: Trying private key: /home/dcrdev/.ssh/id_dsa
debug3: no such identity: /home/dcrdev/.ssh/id_dsa: No such file or directory
debug1: Trying private key: /home/dcrdev/.ssh/id_ecdsa
debug3: no such identity: /home/dcrdev/.ssh/id_ecdsa: No such file or directory
debug1: Trying private key: /home/dcrdev/.ssh/id_ed25519
debug3: no such identity: /home/dcrdev/.ssh/id_ed25519: No such file or
directory
debug2: we did not send a packet, disable method
debug1: No more authentication methods to try.
Permission denied (publickey).

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #32 from Nicolas Chauvet  ---
(In reply to Dominic Robinson from comment #31)
> It doesn't look like the git server has my public key key authorised? Does
> this take a while?
Fixed by forcing genacls.sh script

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Re: [Bug 4647] Review Request: unifi - Ubiquiti UniFi controller

2017-09-12 Thread Stuart Gathman 
On 09/12/2017 04:00 PM, RPM Fusion Bugzilla wrote:
>
> *Comment # 4 
> on bug 4647  from
> Richard  *
> (In reply to Nicolas Chauvet from comment #1 )
> > %ghost is availabe with EL6. I don't understand why you think it was not. > 
> > What is available with newer OS is that it does not require the
> file to be > pre-created (so you need to use touch for EL6).
>
> Ok, I'm digging into this a bit more... If I have to "touch" the file so that
> it exists, doesn't that completely contradict the whole reason for using
> %ghost?
This is what I have for the cjdns package in EPEL, it works for EL6, and
the SPEC file does not use touch.   Maybe what Nicolas meant is that
*something* has to eventually create the file.  In the case of cjdns, it
gets created when starting the service if it doesn't already exist.

%ghost %attr(0600,root,root) %config(missingok,noreplace)
%{_sysconfdir}/cjdroute.conf
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #31 from Dominic Robinson  
---
It doesn't look like the git server has my public key key authorised? Does this
take a while?

debug1: Host 'pkgs.rpmfusion.org' is known and matches the RSA host key.
debug1: Found key in /home/dcrdev/.ssh/known_hosts:1
debug3: send packet: type 21
debug2: set_newkeys: mode 1
debug1: rekey after 4294967296 blocks
debug1: SSH2_MSG_NEWKEYS sent
debug1: expecting SSH2_MSG_NEWKEYS
debug3: receive packet: type 21
debug1: SSH2_MSG_NEWKEYS received
debug2: set_newkeys: mode 0
debug1: rekey after 4294967296 blocks
debug2: key: /home/dcrdev/.ssh/id_rsa (0x55e5989f52b0), agent
debug2: key: /home/dcrdev/.ssh/id_dsa ((nil))
debug2: key: /home/dcrdev/.ssh/id_ecdsa ((nil))
debug2: key: /home/dcrdev/.ssh/id_ed25519 ((nil))
debug3: send packet: type 5
debug3: receive packet: type 6
debug2: service_accept: ssh-userauth
debug1: SSH2_MSG_SERVICE_ACCEPT received
debug3: send packet: type 50
debug3: receive packet: type 51
debug1: Authentications that can continue: publickey
debug3: start over, passed a different list publickey
debug3: preferred
gssapi-keyex,gssapi-with-mic,publickey,keyboard-interactive,password
debug3: authmethod_lookup publickey
debug3: remaining preferred: keyboard-interactive,password
debug3: authmethod_is_enabled publickey
debug1: Next authentication method: publickey
debug1: Offering RSA public key: /home/dcrdev/.ssh/id_rsa
debug3: send_pubkey_test
debug3: send packet: type 50
debug2: we sent a publickey packet, wait for reply
debug3: receive packet: type 51
debug1: Authentications that can continue: publickey
debug1: Trying private key: /home/dcrdev/.ssh/id_dsa
debug3: no such identity: /home/dcrdev/.ssh/id_dsa: No such file or directory
debug1: Trying private key: /home/dcrdev/.ssh/id_ecdsa
debug3: no such identity: /home/dcrdev/.ssh/id_ecdsa: No such file or directory
debug1: Trying private key: /home/dcrdev/.ssh/id_ed25519
debug3: no such identity: /home/dcrdev/.ssh/id_ed25519: No such file or
directory
debug2: we did not send a packet, disable method
debug1: No more authentication methods to try.
Permission denied (publickey).

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

Dominic Robinson  changed:

   What|Removed |Added

 Blocks|3   |
  Flags|needinfo?(development-K9Rvg |needinfo-
   |hem1omxw...@dcrdev.com) |


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3
[Bug 3] Tracker: Packages under review.
-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4647] Review Request: unifi - Ubiquiti UniFi controller

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647

--- Comment #4 from Richard  ---
(In reply to Nicolas Chauvet from comment #1)
> %ghost is availabe with EL6. I don't understand why you think it was not.
> What is available with newer OS is that it does not require the file to be
> pre-created (so you need to use touch for EL6).

Ok, I'm digging into this a bit more... If I have to "touch" the file so that
it exists, doesn't that completely contradict the whole reason for using
%ghost?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Re: FTBFS for f27

2017-09-12 Thread Ben Rosser 
On Mon, Sep 11, 2017 at 11:32 AM, Andrea Musuruane  wrote:
> BTW, to have a complete fs-uae experience we still miss fs-uae-launcher
> (#4028) and fs-uae-arcade (#4029). I'm available to swap reviews.
>
> Regards,
>
> Andrea

I'd be happy to review both of these.

Are you willing to review
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4639 (dfhack) in
exchange? It's a somewhat complex package, so if not, that's okay.

Ben Rosser
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4639] dfhack - Memory hacking library for Dwarf Fortress and a set of tools that use it

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4639

Ben Rosser  changed:

   What|Removed |Added

 Blocks||2


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=2
[Bug 2] Tracker: New packages awaiting review
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4152] Review Request: vdr-softhddevice-openglosd - A software and GPU emulated HD output device plugin for VDR

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4152

--- Comment #4 from mgans...@online.de  ---
Thanks for reviewing the ticket.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4647] Review Request: unifi - Ubiquiti UniFi controller

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647

--- Comment #3 from Richard  ---
I guess I can create a bash script and put it in /usr/sbin...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4647] Review Request: unifi - Ubiquiti UniFi controller

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647

--- Comment #2 from Richard  ---
(In reply to Nicolas Chauvet from comment #1)
> I expect you talked about the nonfree section since it's proprietary
> (the namespace is set as appropriate).
> 
> 
> You can use NoSource: 0 (After Source0) instead of using with conditional to
> avoid producing src.rpm with the Source0 in it. (it will be named nosrc.rpm
> instead).

I changed the conditional to by default have the source within the SRPM.
Initially I had to default the other way until I was given permission to
distribute. I plan to remove the conditionals but some may find it useful to
upgrade sooner than I would update the package in RPM Fusion so it still has
some value.


> About pre-built binaries. 
> You only need to use:
> %define debug_package %{nil}
> %define __strip /bin/true
> 
> Please avoid to redefine  __os_install_post

I'm sure I added that for a reason but I can't remember at this point. I'll
test without it.


> Please drop # Selinux versioned requires

I got this from here:
https://fedoraproject.org/wiki/SELinux_Policy_Modules_Packaging_Draft#Runtime_Dependencies

I understand it's a draft, but does it not apply?


> ExclusiveArch:  x86_64 armv7hl
> 
> Uses: Requires: java > 1:1.8.0
> https://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires

Updated to:
Requires:   java--headless >= 1:1.8.0


> Why Conflicts: unifi-controller ? Can this be avoided ?

There is another person providing unofficial packages which installs into /opt
and I don't want anyone to accidentally install both packages.


> %ghost is availabe with EL6. I don't understand why you think it was not.
> What is available with newer OS is that it does not require the file to be
> pre-created (so you need to use touch for EL6).

I'm sure I tried it and it didn't seem to work. I'll try again.


> %pre , you are probably missing the proper consition to only move the logs
> files only on upgrade on installation. Why this move is needed probably need
> to be stated as a comment.

In one of my earlier versions of the packages I didn't realize the logs were
being written to /usr/share/unifi/logs hence the migration logic. Since the
files won't be deleted on package removal I don't want to depend on only
running the script on upgrade.


> It doesn't look acceptable to edit the selinux configuration on post
> installation. First you need not to assume selinux might not run at all on
> the end-user system. Second you should better the needed changes to the
> targeted policy instead.

So if I move that to install the policy changes will make it into the resultant
RPM? I'm assuming restorecon still needs to be in %post?


> execstack need to be modified in %install step, not %post or you will break
> rpm -V

Again, same question. This will make it into the RPM? 


> About the init_t in permissive, this is not at all acceptable either. You
> are really changing a system wide setting in the back of end-users. 
> https://bugzilla.redhat.com/show_bug.cgi?id=1358476#c20

What's the alternative? Seems better than telling everyone they have to run
selinux in permissive mode.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #30 from Nicolas Chauvet  ---
okay, so this package is formally APPROVED.

I will give the ownership to you.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #29 from Dominic Robinson  
---
(In reply to Nicolas Chauvet from comment #28)
> (In reply to Dominic Robinson from comment #26)
> > Also bit confused - is this package approved, I note the fedora-review +
> > flag and also the repository having been created here:
> Yep, seems like this package was mass imported along the other ones from
> Mohamed.
> I will transfer the ownership to you once this package got formally approved
> can you remind your RPM Fusion FAS account name ?

Yes it's "dcrdev" .

(In reply to Nicolas Chauvet from comment #27)
> (In reply to Dominic Robinson from comment #25)
> > I hadn't planned on changing it -
> > 
> > I mean might be a problem if I reuse this spec for other addons, but I don't
> > see an issue in the context of this addon and makes things neater.
> 
> Well, this is not the case from a RPM Fusion rel-eng perspective.
> If ever the aname is changed, which might happen if the package get fixed ,
> the package name will be automatically changed. This is not possible with
> the infra where koji expect the spec file to be named according to the git
> module name.
> As a consequence, the build will fails, but it may requires additional steps
> to cure this issue.
> 
> For example, upstream may consider to change inputstream.adaptive to
> inputstream_adaptive. So the point of using a variable in this case doesn't
> hold.

OK makes sense - I'll hardcode it to "kodi-inputstream-adaptive" . I'll kick
off a build this evening, as I don't have access to my build system right now.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #28 from Nicolas Chauvet  ---
(In reply to Dominic Robinson from comment #26)
> Also bit confused - is this package approved, I note the fedora-review +
> flag and also the repository having been created here:
Yep, seems like this package was mass imported along the other ones from
Mohamed.
I will transfer the ownership to you once this package got formally approved
can you remind your RPM Fusion FAS account name ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3176] Review Request: lpf-mscore-tahoma-fonts - Microsoft core Tahoma fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3176

--- Comment #6 from Sérgio Basto  ---
now it is built in koji

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #27 from Nicolas Chauvet  ---
(In reply to Dominic Robinson from comment #25)
> I hadn't planned on changing it -
> 
> I mean might be a problem if I reuse this spec for other addons, but I don't
> see an issue in the context of this addon and makes things neater.

Well, this is not the case from a RPM Fusion rel-eng perspective.
If ever the aname is changed, which might happen if the package get fixed , the
package name will be automatically changed. This is not possible with the infra
where koji expect the spec file to be named according to the git module name.
As a consequence, the build will fails, but it may requires additional steps to
cure this issue.

For example, upstream may consider to change inputstream.adaptive to
inputstream_adaptive. So the point of using a variable in this case doesn't
hold.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #26 from Dominic Robinson  
---
Also bit confused - is this package approved, I note the fedora-review + flag
and also the repository having been created here:
https://admin.rpmfusion.org/pkgdb/package/free/kodi-inputstream-adaptive/ .

Wasn't I suppose to be the one who requests the package, as it stands I'm not
the maintainer of that repository?

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #25 from Dominic Robinson  
---
I hadn't planned on changing it -

I mean might be a problem if I reuse this spec for other addons, but I don't
see an issue in the context of this addon and makes things neater.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3177] Review Request: lpf-mscore-fonts - Microsoft core TrueType fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3177

Sérgio Basto  changed:

   What|Removed |Added

 CC||ser...@serjux.com

--- Comment #6 from Sérgio Basto  ---
this was built in koji yesterday

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3176] Review Request: lpf-mscore-tahoma-fonts - Microsoft core Tahoma fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3176

--- Comment #5 from Sérgio Basto  ---
sorry not yet built in koji , I will build it soon

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3176] Review Request: lpf-mscore-tahoma-fonts - Microsoft core Tahoma fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3176

Sérgio Basto  changed:

   What|Removed |Added

 CC||ser...@serjux.com

--- Comment #4 from Sérgio Basto  ---
built in koji

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

Nicolas Chauvet  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+

--- Comment #24 from Nicolas Chauvet  ---
(In reply to Dominic Robinson from comment #23)
> I don't see any problem with that per say , but I know for a fact that this
> can only be built against kodi >= 17; for future versions sure.
> 
> So to clear up my limited understanding of koji, how would this work
> exactly? If a koji_version macro were to be defined in the kodi-devel
> package, how would that get passed to this package?
Sorry, I meant a %kodi_version macro. 

It would be defined in the kodi-devel package (
echo %kodi_version %{version} > %{buildroot}%{rpmmacrodir}/macros.kodi
)

Then when plugins would build, they would have:
Requires: kodi >= %{kodi_version}


Anyway, it seems a minor issue and not particular to this plugins. this can be
fixed and coordinated later. 

Do you plan to fix the name with modified macro ? This looks really scary.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

--- Comment #23 from Dominic Robinson  
---
I don't see any problem with that per say , but I know for a fact that this can
only be built against kodi >= 17; for future versions sure.

So to clear up my limited understanding of koji, how would this work exactly?
If a koji_version macro were to be defined in the kodi-devel package, how would
that get passed to this package?

Is it a case of:
- Dependent packages are being built in a single chroot a la mock chain.
- Therefore once koji_version is set, it is exported before all subsequent
builds within that chroot.

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3177] Review Request: lpf-mscore-fonts - Microsoft core TrueType fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3177

Nicolas Chauvet  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4152] Review Request: vdr-softhddevice-openglosd - A software and GPU emulated HD output device plugin for VDR

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4152

Nicolas Chauvet  changed:

   What|Removed |Added

 Blocks|3   |4
  Flags|fedora-review?  |fedora-review+

--- Comment #3 from Nicolas Chauvet  ---
koji scratch build succeeded for all arches:
http://koji.rpmfusion.org/koji/taskinfo?taskID=167123

Package is APPROVED


Referenced Bugs:

https://bugzilla.rpmfusion.org/show_bug.cgi?id=3
[Bug 3] Tracker: Packages under review.
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4
[Bug 4] Tracker: Accepted packages.
-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 3176] Review Request: lpf-mscore-tahoma-fonts - Microsoft core Tahoma fonts for better Windows Compatibility

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=3176

Nicolas Chauvet  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4532] Review request: kodi-inputstream-adaptive - Adaptive file addon for Kodi's InputStream interface

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4532

Nicolas Chauvet  changed:

   What|Removed |Added

  Flags||needinfo?(development-K9Rvg
   ||hem1omxw...@dcrdev.com)

--- Comment #22 from Nicolas Chauvet  ---
Ping ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4640] Review request: slowmoVideo - Creates slow-motion videos from your footage

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4640

Nicolas Chauvet  changed:

   What|Removed |Added

 CC||meta...@gmail.com
  Flags||needinfo?(meta...@gmail.com
   ||)

--- Comment #3 from Nicolas Chauvet  ---
Ping ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4634] Review request: kodi-peripheral-joystick - Kodi Joystick Report

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4634

Nicolas Chauvet  changed:

   What|Removed |Added

 CC||wberr...@gmail.com
  Flags||fedora-review?,
   ||needinfo?(wberr...@gmail.co
   ||m)

--- Comment #5 from Nicolas Chauvet  ---
Ping ?

-- 
You are receiving this mail because:
You are on the CC list for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

[Bug 4647] Review Request: unifi - Ubiquiti UniFi controller

2017-09-12 Thread RPM Fusion Bugzilla 
https://bugzilla.rpmfusion.org/show_bug.cgi?id=4647

--- Comment #1 from Nicolas Chauvet  ---
I expect you talked about the nonfree section since it's proprietary
(the namespace is set as appropriate).


You can use NoSource: 0 (After Source0) instead of using with conditional to
avoid producing src.rpm with the Source0 in it. (it will be named nosrc.rpm
instead).

Unfortunately, it will not work at all with mock since you expect the whole
content to be provided and even less with koji since it expect no internet
access (you might have access to the cache source at least).
To be tested on scratch build...

The other point is that you cannot have a different source/binary content. So
koji will now allows to build the binary with the .src.rpm once, then the
"nosrc.rpm" on the second build and only keep the former binary and the later
no source rpm.


About pre-built binaries. 
You only need to use:
%define debug_package %{nil}
%define __strip /bin/true

Please avoid to redefine  __os_install_post

Please drop # Selinux versioned requires

ExclusiveArch:  x86_64 armv7hl

Uses: Requires: java > 1:1.8.0
https://fedoraproject.org/wiki/Packaging:Java#BuildRequires_and_Requires

Why Conflicts: unifi-controller ? Can this be avoided ?

%ghost is availabe with EL6. I don't understand why you think it was not. What
is available with newer OS is that it does not require the file to be
pre-created (so you need to use touch for EL6).


%pre , you are probably missing the proper consition to only move the logs
files only on upgrade on installation. Why this move is needed probably need to
be stated as a comment.


It doesn't look acceptable to edit the selinux configuration on post
installation. First you need not to assume selinux might not run at all on the
end-user system. Second you should better the needed changes to the targeted
policy instead.

execstack need to be modified in %install step, not %post or you will break rpm
-V

About the init_t in permissive, this is not at all acceptable either. You are
really changing a system wide setting in the back of end-users. 
https://bugzilla.redhat.com/show_bug.cgi?id=1358476#c20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org

Re: FTBFS for f27

2017-09-12 Thread Leigh Scott 
sonic-visualiser-freeworld can be retired as mad was moved to fedora, 
sonic-visualiser now has mad support
The same applies to sox-plugins-freeworld.

As for xine-plugin, it seems to have been built

https://koji.rpmfusion.org/koji/buildinfo?buildID=4994

I believe it should be retired once Firefox-52 ESR support ends
___
rpmfusion-developers mailing list -- rpmfusion-developers@lists.rpmfusion.org
To unsubscribe send an email to rpmfusion-developers-le...@lists.rpmfusion.org