[Bug 554464] Review Request: python-pebl - Python Environment for Bayesian Learning
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=554464
Thomas Spura <tomspur(a)fedoraproject.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag|fedora-review? |fedora-review+
--- Comment #18 from Thomas Spura <tomspur(a)fedoraproject.org> 2011-10-09 12:14:15 EDT ---
(In reply to comment #17)
> (In reply to comment #16)
> > - Disabling the internal dependency generator is not needed, when using:
> >
> > https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering...
>
> OK, I fixed that (I used the first option).
OK
> > - For a failing test suite you could append "| :" so at least anything else is
> > checked for now. (Maybe it will completely work, when you also add the
> > dependencies)
>
> As it turned out, this is a good thing because it uncovered new failed unit
> tests.
> I noted how many tests are failing and added a link to the upstream bug report.
Thanks.
Some are failing because of a missing "testdata5m.txt", when generating the tar
ball upstream seems to not include that here.
Some others seem to cause failures in pydot:
File "/usr/lib/python2.7/site-packages/pydot.py", line 682, in <lambda>
self.__setattr__('get_'+attr, lambda a=attr : self.__get_attribute__(a))
File "/usr/lib/python2.7/site-packages/pydot.py", line 594, in
__get_attribute__
defaults = self.get_parent_graph().get_node( default_node_name )
This also makes it impossible to run the tutorials from:
http://packages.python.org/pebl/tutorial.html
Maybe it would be best to file a bug against pydot...
> Links to the updated files:
> Spec URL: http://tadej.fedorapeople.org/python-pebl.spec
> SRPM URL: http://tadej.fedorapeople.org/python-pebl-1.0.2-6.fc15.src.rpm
>
> I also did a scratch build of the package in koji and the results are here:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=3416390
Thanks.
______________________________________________________
The package/spec looks fine, but I'd not import this, untill the tutorials are
working...
Filed as a bug against pydot with a patch (see depends on bug).
______________________________________________________
APPROVED
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 6 months
[Bug 554464] Review Request: python-pebl - Python Environment for Bayesian Learning
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=554464
--- Comment #16 from Thomas Spura <tomspur(a)fedoraproject.org> 2011-10-09 06:21:43 EDT ---
(In reply to comment #15)
> I applied the Toshio's patch to fix most of the warnings when building the
> Sphinx documentation.
OK
> I put a new version of the .spec file and .src.rpm to
> http://tadej.fedorapeople.org/.
Can you please provide SPEC/SRPM links like in comment #0 next time?
Thanks.
> I also did a scratch build of the package in koji and the results are here:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=3414470
OK
I get more rpmlint outputs:
$ rpmlint ~/rpmbuild/SRPMS/python-pebl-1.0.2-5.fc15.src.rpm
~/rpmbuild/RPMS/x86_64/python-pebl-*
python-pebl.src: W: spelling-error %description -l en_US interventional ->
intervention, international, Internationale
python-pebl.src: W: invalid-url Source0:
http://pebl-project.googlecode.com/files/pebl-1.0.2.tar.gz HTTP Error 404: Not
Found
python-pebl.x86_64: E: explicit-lib-dependency python-matplotlib
python-pebl.x86_64: W: spelling-error %description -l en_US interventional ->
intervention, international, Internationale
python-pebl.x86_64: W: no-manual-page-for-binary pebl
3 packages and 0 specfiles checked; 1 errors, 4 warnings.
But all ignoreable/false positives.
e.g. spectool works, so the link to the source is good.
- Disabling the internal dependency generator is not needed, when using:
https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering...
or:
https://fedoraproject.org/wiki/User:Tibbs/AutoProvidesAndRequiresFilterin...
(depends, where you want to branch later on)
- For a failing test suite you could append "| :" so at least anything else is
checked for now. (Maybe it will completely work, when you also add the
dependencies)
- ipython1 should be dead, I hope upstream will adopt the new ipython -.-
_____________________________________________
Will approve, when the dependency generator looks fine, test suite is "should".
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 6 months
[Bug 554464] Review Request: python-pebl - Python Environment for Bayesian Learning
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=554464
--- Comment #15 from Tadej Janež <tadej.janez(a)tadej.hicsalta.si> 2011-10-08 16:29:49 EDT ---
(In reply to comment #14)
> Any news here?
> Is doc generation working now?
Sorry about the long delay, this completely fell out of my radar.
I applied the Toshio's patch to fix most of the warnings when building the
Sphinx documentation.
I put a new version of the .spec file and .src.rpm to
http://tadej.fedorapeople.org/.
I also did a scratch build of the package in koji and the results are here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3414470
$rpmlint python-pebl-1.0.2-5.fc15.src.rpm
python-pebl.src: W: spelling-error %description -l en_US interventional ->
intervention, international, Internationale
python-pebl.src: W: invalid-url Source0:
http://pebl-project.googlecode.com/files/pebl-1.0.2.tar.gz HTTP Error 404: Not
Found
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
I think the first warning can be ignored. The second, however, puzzles me,
because I can open the same link in Firefox and download Pebl?
$ rpmlint python-pebl
python-pebl.x86_64: E: explicit-lib-dependency python-matplotlib
python-pebl.x86_64: W: spelling-error %description -l en_US interventional ->
intervention, international, Internationale
python-pebl.x86_64: W: no-manual-page-for-binary pebl
1 packages and 0 specfiles checked; 1 errors, 2 warnings.
The first error should be OK, as discussed in comment #4. The two warnings can
be ignored for now.
Regards,
Tadej
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 6 months
[Bug 744416] New: Review Request: libisofs1 - Library to create ISO 9660 disk images
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: libisofs1 - Library to create ISO 9660 disk images
https://bugzilla.redhat.com/show_bug.cgi?id=744416
Summary: Review Request: libisofs1 - Library to create ISO 9660
disk images
Product: Fedora EPEL
Version: el6
Platform: Unspecified
OS/Version: Unspecified
Status: NEW
Severity: unspecified
Priority: unspecified
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: redhat-bugzilla(a)linuxnetz.de
QAContact: extras-qa(a)fedoraproject.org
CC: fedora-package-review(a)redhat.com
Estimated Hours: 0.0
Classification: Fedora
Story Points: ---
Type: ---
Spec URL: http://labs.linuxnetz.de/bugzilla/libisofs1.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/libisofs1-1.1.6-2.src.rpm
Description:
Libisofs is a library to create an ISO-9660 filesystem and supports
extensions like RockRidge or Joliet. It is also a full featured
ISO-9660 editor, allowing you to modify an ISO image or multisession
disc, including file addition or removal, change of file names and
attributes etc. It supports the extension AAIP which allows to store
ACLs and xattr in ISO-9660 filesystems as well. As it is linked with
zlib, it supports zisofs compression, too.
RHEL 6 is shipping libisofs-0.6.32-1.el6, but Red Hat doesn't seem to
be interested to update libisofs to a more recent version, which would
be required to build libisoburn/xorriso for RHEL 6. This package can be
installed in parallel with the regular RHEL 6 package due to renaming.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 6 months