Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 600529 - Review Request: R-rjags - Bayesian graphical models using MCMC
Review Request: R-rjags - Bayesian graphical models using MCMC
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
low Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On: 600467
  Show dependency treegraph
Reported: 2010-06-04 18:31 EDT by chris desjardins
Modified: 2011-01-19 17:47 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2011-01-19 17:47:51 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description chris desjardins 2010-06-04 18:31:54 EDT
Spec URL: http://dl.dropbox.com/u/1501309/Fedora/R-rjags.spec
SRPM URL: http://dl.dropbox.com/u/1501309/Fedora/R-rjags-
Bayesian graphical models using MCMC. rjags is an R interface to the JAGS MCMC library
Comment 1 Pierre-YvesChibon 2010-06-04 19:01:34 EDT
As a quick look at the spec:
- The URL is not correct
- The BuildRequires to tetex-latex has to be updated
- The Requires for post and postun can be removed
- you might want to change the Requires from R to R-core 
- You have a double dependency on R in your BuildRequires 
- The comment on the %install is incorrect (also in %files)

The rest look correct, I will do a deeper pre-review (as I can not sponsor you) later.
Comment 2 chris desjardins 2010-06-04 19:26:48 EDT
I have made the changes to my *.spec file and have updated new versions of the *.spec and *.src.rpm files. They can be obtained by clicking my original urls in the description box.
Comment 3 chris desjardins 2010-06-04 19:28:12 EDT
I will update the *.spec file with new version and a changelog. Sorry!
Comment 4 chris desjardins 2010-06-04 20:09:47 EDT
Here is the new *.spec file and the new *src.rpm addressing Pierre's issues.

Comment 5 Pierre-YvesChibon 2010-06-05 08:41:57 EDT
* Latest version packaged
Target is already in the folder, no need to re-download it
7b3a345ad25b859c053111e607c8caa312a943ae  rjags_2.1.0-4.tar.gz
7b3a345ad25b859c053111e607c8caa312a943ae  /home/pierrey/rpmbuild/SOURCES/rjags_2.1.0-4.tar.gz
* sha1sum are equals
* All required Requires are present
* All required BuildRequires are present
* The macro %check is present
* There is 1 %dir
  %dir is OK
* There is 2 %doc
  %doc is OK
* The rpm installed in _libdir
* The rpm uses %{_libdir} and is arch
* The rpm seems to have the required element in %install
rpmbuild -ba /home/pierrey/rpmbuild/SPECS/R-rjags.spec > R-rjags.spec-build.log 2>&1 
! Does not build properly

I believe you are missing at least one BuildRequires
Comment 6 chris desjardins 2010-06-05 09:56:37 EDT
This should build now.


However, this package requires a package not yet packaged in Fedora. So I think unless you're willing to try and build JAGS[1] then it won't build as it's an interface between R and JAGS. 

[1] https://bugzilla.redhat.com/show_bug.cgi?id=600467

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