Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: perl-SystemPerl - SystemPerl Perl module
https://bugzilla.redhat.com/show_bug.cgi?id=478759
Summary: Review Request: perl-SystemPerl - SystemPerl Perl module Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: cgoorah@yahoo.com.au QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.300-1.fc10.src.rpm Description: SystemPerl is a version of the SystemC language. It is designed to expand text so that needless repetition in the language is minimized. By using sp_preproc, SystemPerl files can be expanded into C++ files at compile time, or expanded in place to make them valid stand-alone SystemC files.
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=478759
Chitlesh GOORAH cgoorah@yahoo.com.au changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |468516
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=478759
--- Comment #1 from Chris Weyl cweyl@alumni.drew.edu 2009-01-04 18:42:15 EDT --- Koji (FAILURE) http://koji.fedoraproject.org/koji/taskinfo?taskID=1032161
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=478759
Chitlesh GOORAH cgoorah@yahoo.com.au changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |476386
--- Comment #2 from Chitlesh GOORAH cgoorah@yahoo.com.au 2009-01-05 17:30:10 EDT ---
Yes in accordance to the logs :
DEBUG util.py:250: No Package Found for perl(Verilog::Getopt) >= 2.211 DEBUG util.py:250: No Package Found for perl(Verilog::Netlist) >= 2.315
you will need perl-Verilog to build this rpm.
perl-Verilog recently was approved. https://bugzilla.redhat.com/show_bug.cgi?id=476386
It is just a matter of time till perl-Verilog hits rawhide. I should have blocked this review against perl-Verilog.
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=478759
Lane dirjud@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dirjud@gmail.com
--- Comment #3 from Lane dirjud@gmail.com 2009-01-07 05:10:27 EDT --- Chitlesh,
I am experiencing a compatibility problem with the location of the perl-SystemPerl-devel files. Verilator (https://bugzilla.redhat.com/show_bug.cgi?id=468516) compiles in some of the *.cpp files from the systemperl src directory when it creates a simulation model. When you compile verilator, you give it the path to the systemperl installation, and when you use verilator to create a simulation model, it looks for those *.cpp files in the src/*.cpp directory. Because the systemperl spec file is moving these *.cpp files into the /usr/include/SystemPerl directory, there is no way for verilator to find them. They would need to go in the /usr/include/SystemPerl/src directory for verilator to be able to find them. Given these files are not used for linking but are used for actual compilation, perhaps they should go into the %{_datadir}/src directory.
Lane
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=478759
--- Comment #4 from Lane dirjud@gmail.com 2009-01-07 05:17:33 EDT --- If I change the SystemPerl spec file to:
# -devel package and support for SystemC %{__mkdir} -p $RPM_BUILD_ROOT%{_includedir}/%{name}/src %{__install} -pm 0644 src/*.h $RPM_BUILD_ROOT%{_includedir}/%{name}/src/ %{__install} -pm 0644 src/*.cpp $RPM_BUILD_ROOT%{_includedir}/%{name}/src/
and then set
SYSTEMPERL=%{_includedir}/SystemPerl
in the verilator.spec file, then I can get verilator to pickup the systemperl files it needs to create simulation models correctly.
I do not know enough of the fedora conventions to know where these cpp files *should* go, but wherever they end up, they need to be need to be in a directory called */src/
Lane
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=478759
wsnyder@wsnyder.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |wsnyder@wsnyder.org
Bug 478759 depends on bug 476386, which changed state.
Bug 476386 Summary: Review Request: perl-verilog - Verilog parsing routines https://bugzilla.redhat.com/show_bug.cgi?id=476386
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
--- Comment #5 from wsnyder@wsnyder.org 2009-01-28 20:38:08 EDT --- Chitlesh notified me of this thread.
The src/ prefix is because many users have multiple versions of SystemPerl installed (generally in a repository) and just point to the one they want.
Lane has a the right solution for the present version, as makefiles etc also have the src/ path hardcoded. If there's a strong objection to the extra src/ in the path, I can add a new envvariable that will set the location.
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=478759
--- Comment #6 from Chitlesh GOORAH cgoorah@yahoo.com.au 2009-01-31 19:59:16 EDT --- (In reply to comment #5)
Chitlesh notified me of this thread.
The src/ prefix is because many users have multiple versions of SystemPerl installed (generally in a repository) and just point to the one they want.
Lane has a the right solution for the present version, as makefiles etc also have the src/ path hardcoded. If there's a strong objection to the extra src/ in the path, I can add a new envvariable that will set the location.
Hello Wilson,
From my point of view, in order to ease the installation of multiple version of
SystemPerl, those *.cpp should rather be placed into -- /usr/include/perl-SystemPerl - for the distribution supported package -- /usr/include/perl-SystemPerl-$version - for parallel installations
I would welcome an envvariable.
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=478759
--- Comment #7 from Chitlesh GOORAH chitlesh@gmail.com 2009-03-06 14:51:12 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.310-1.fc10.src.rpm
I've added the src directory to fix the build on verilator. A small patch was sent to Wilson to fix the build on the latest bison
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=478759
Brennan Ashton bashton@brennanashton.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |bashton@brennanashton.com
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=478759
Brennan Ashton bashton@brennanashton.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |bashton@brennanashton.com
--- Comment #8 from Brennan Ashton bashton@brennanashton.com 2009-03-08 05:38:53 EDT --- rpmlint is showing this error: perl-SystemPerl.i386: E: useless-provides perl(SystemC::Netlist::Module)
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=478759
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review?
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=478759
--- Comment #9 from Chitlesh GOORAH chitlesh@gmail.com 2009-03-09 15:46:05 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.310-2.fc10.src.rpm
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=478759
--- Comment #10 from Chitlesh GOORAH chitlesh@gmail.com 2009-03-25 15:07:43 EDT --- Ping ?
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=478759
Ralf Corsepius rc040203@freenet.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rc040203@freenet.de
--- Comment #11 from Ralf Corsepius rc040203@freenet.de 2009-03-27 01:18:20 EDT --- Just an observation:
The emacs support files' packaging seems suspicious to me # rpm -qlp emacs-systemc-mode-1.310-2.fc11.x86_64.rpm /usr/share/emacs/site-lisp/site-start.d/systemc-mode.el /usr/share/emacs/site-lisp/systemc-mode.el /usr/share/emacs/site-lisp/systemc-mode.elc
When comparing this to my FC10 installation: # rpm -qf /usr/share/emacs/site-lisp subversion-1.5.4-3.x86_64 libidn-0.6.14-8.x86_64 desktop-file-utils-0.15-3.fc10.x86_64 autoconf-2.63-1.fc10.noarch libidn-0.6.14-8.i386 rpmdevtools-7.0-1.fc10.noarch cmake-2.6.3-3.fc10.x86_64 ... # rpm -qf /usr/share/emacs/site-lisp/site-start.d rpmdevtools-7.0-1.fc10.noarch
... I am inclined to think, directory ownerships aren't handled correctly. You likely need to own these packages.
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=478759
--- Comment #12 from wsnyder@wsnyder.org 2009-03-27 14:55:21 EDT ---
/usr/share/emacs/site-lisp/systemc-mode.el /usr/share/emacs/site-lisp/systemc-mode.elc
Don't bother installing these files. Emacs 22.2 and above breaks this file and it's proven almost impossible to keep them working on both 21.* and 22.*, so the next version of SystemPerl will no longer install it.
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=478759
--- Comment #13 from wsnyder@wsnyder.org 2009-03-27 15:35:41 EDT ---
/usr/share/emacs/site-lisp/systemc-mode.el /usr/share/emacs/site-lisp/systemc-mode.elc
Don't bother installing these files. Emacs 22.2 and above breaks this file and it's proven almost impossible to keep them working on both 21.* and 22.*, so the next version of SystemPerl will no longer install it.
I take that back, a hook is still needed for the .sp file extension, so something still needs to exist. Thus I've created a systemc-mode.el shell that simply calls c++-mode. (Grab it from CPAN mid next week.)
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=478759
--- Comment #14 from wsnyder@wsnyder.org 2009-03-28 11:14:32 EDT --- FYI SystemPerl 1.311 fixes the emacs issue I mentioned (not permissions - that's at your end.)
Also Verilator 3.702 allows you to set and compile SYSTEMPERL_INCLUDE into verilator so the src/ suffix hack (see the earlier comments here) isn't needed.
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=478759
--- Comment #15 from Chitlesh GOORAH chitlesh@gmail.com 2009-03-29 12:09:30 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.311-1.fc10.src.rpm
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=478759
--- Comment #16 from Chitlesh GOORAH chitlesh@gmail.com 2009-03-29 12:11:24 EDT --- Thanks Wilson for this quick update.
Brennan as from now on, please point SYSTEMPERL_INCLUDE to /usr/include/perl-SystemPerl.
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=478759
--- Comment #17 from Chitlesh GOORAH chitlesh@gmail.com 2009-05-14 19:39:22 EDT --- ping Brennan ?
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=478759
--- Comment #18 from Chitlesh GOORAH chitlesh@gmail.com 2009-05-21 12:23:57 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemPerl-1.320-1.fc10.src.rpm
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=478759
--- Comment #19 from Brennan Ashton bashton@brennanashton.com 2009-05-21 19:14:27 EDT --- I will evaluate this new package tonight.
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=478759
--- Comment #20 from Chitlesh GOORAH chitlesh@gmail.com 2009-05-22 05:13:16 EDT --- Thanks you will need the last perl-Verilog, already built on koji. Please pull the rpm from koji if perl-Verilog has not yet hit the mirrors.
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=478759
--- Comment #21 from Brennan Ashton bashton@brennanashton.com 2009-05-25 18:41:44 EDT --- I have some concerns about if this is passing its buildtime tests:
t/00_pod.t ......... ok t/01_manifest.t .... ok t/02_help.t ........ ok t/03_spaces.t ...... ok t/10_template.t .... ok t/20_parser.t ...... ok t/30_netlist.t ..... ok t/32_example.t ..... ok t/34_notfound.t .... ok t/40_coverage.t .... ok t/42_itemkey.t ..... ok t/44_makecheck.t ... ok t/60_spp_pre.t ..... 1/6 Integer overflow in hexadecimal number at /home/makerpm/rpmbuild/BUILD/SystemPerl-1.320/blib/lib/SystemC/Netlist/CoverPoint.pm line 473. Integer overflow in hexadecimal number at /home/makerpm/rpmbuild/BUILD/SystemPerl-1.320/blib/lib/SystemC/Netlist/CoverPoint.pm line 473. t/60_spp_pre.t ..... ok t/62_spp_inline.t .. 1/2 Integer overflow in hexadecimal number at /home/makerpm/rpmbuild/BUILD/SystemPerl-1.320/blib/lib/SystemC/Netlist/CoverPoint.pm line 473. Integer overflow in hexadecimal number at /home/makerpm/rpmbuild/BUILD/SystemPerl-1.320/blib/lib/SystemC/Netlist/CoverPoint.pm line 473. t/62_spp_inline.t .. ok t/70_trace.t ....... ok t/80_gcc.t ......... ok t/85_scl.t ......... ok t/90_ncsc_run.t .... ok All tests successful.
This is on a F11 machine with this per-Verlog http://koji.fedoraproject.org/koji/buildinfo?buildID=103021
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=478759
--- Comment #22 from wsnyder@wsnyder.org 2009-05-25 19:04:29 EDT --- The warnings are due to a test that seems to not be 32-bit compatible. I'll fix it - but its only the test that needs to change, and since the tests don't get installed it should be fine to use this version.
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=478759
--- Comment #23 from Chitlesh GOORAH chitlesh@gmail.com 2009-05-26 07:12:33 EDT ---
Brennan, can you continue with the review ?
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=478759
--- Comment #24 from Chitlesh GOORAH chitlesh@gmail.com 2009-06-04 04:32:10 EDT --- ping Brennan ?
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=478759
--- Comment #25 from Lane dirjud@gmail.com 2009-06-09 15:08:35 EDT --- I tried building this and it failed saying it required perl-Verilog 3.2. You might want to update this version in the build dependencies list. Also, is perl-Verilog 3.2 rpm available? I only saw 3.12 in rawhide. Maybe I missed it.
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=478759
Brennan Ashton bashton@brennanashton.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #26 from Brennan Ashton bashton@brennanashton.com 2009-06-09 15:37:22 EDT --- [PASS] source files match upstream [PASS] package meets naming and versioning guidelines. [PASS] specfile is properly named [PASS] dist tag is present. [PASS] build root is correct. [PASS] license field matches the actual license. [PASS] license is open source-compatible. [PASS] latest version is being packaged. [PASS] BuildRequires are proper. [PASS] %clean is present. [N/A] package builds in mock. Building with a koji build of perl-Verlog
[PASS] package installs properly. debuginfo package looks complete. [PASS] %check is present and all tests pass: [PASS] owns the directories it creates. [PASS] doesn't own any directories it shouldn't. [PASS] no duplicates in %files. [PASS] file permissions are appropriate. [PASS] no scriptlets present. [PASS] code, not content. [PASS] %docs are not necessary for the proper functioning of the package.
I Approve with the perl-Verlog in koji. Lane is right, you should add the new version requirement of perl-Verlog to the spec. After this go ahead with CVS request.
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=478759
Chitlesh GOORAH chitlesh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?
--- Comment #27 from Chitlesh GOORAH chitlesh@gmail.com 2009-06-10 17:13:38 EDT --- (In reply to comment #25)
I tried building this and it failed saying it required perl-Verilog 3.2. You might want to update this version in the build dependencies list. Also, is perl-Verilog 3.2 rpm available? I only saw 3.12 in rawhide. Maybe I missed it.
F-9 and F-10 already have perl-Verilog 3.200. But perl-SystemC-Vregs failed to build for rawhide and F-11. https://admin.fedoraproject.org/updates/search/perl-Verilog
I'll look at it tomorrow before requesting cvs access for this package.
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=478759
Chitlesh GOORAH chitlesh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo? |
--- Comment #28 from Chitlesh GOORAH chitlesh@gmail.com 2009-06-10 19:08:58 EDT --- perl-Verilog-3.210-1.fc11 was missing in F-11 buildroot and ultimately the chainbuild failed. Aticket was filed https://fedorahosted.org/rel-eng/ticket/1921
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=478759
--- Comment #29 from Chitlesh GOORAH chitlesh@gmail.com 2009-06-11 05:20:57 EDT --- New Package CVS Request ======================= Package Name: perl-SystemPerl Short Description: SystemPerl Perl module Owner chitlesh Branches: F-10 F11 EL-5
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=478759
Chitlesh GOORAH chitlesh@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
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=478759
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #30 from Jason Tibbitts tibbs@math.uh.edu 2009-06-11 13:02:14 EDT --- CVS done.
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=478759
--- Comment #31 from Fedora Update System updates@fedoraproject.org 2009-06-11 15:20:56 EDT --- perl-SystemPerl-1.321-1.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/perl-SystemPerl-1.321-1.fc10
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=478759
--- Comment #32 from Fedora Update System updates@fedoraproject.org 2009-06-11 15:21:33 EDT --- perl-SystemPerl-1.321-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-SystemPerl-1.321-1.fc11
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=478759
--- Comment #33 from Fedora Update System updates@fedoraproject.org 2009-06-15 21:37:18 EDT --- perl-SystemPerl-1.321-1.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
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=478759
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Fixed In Version| |1.321-1.fc11 Resolution| |NEXTRELEASE
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=478759
--- Comment #34 from Fedora Update System updates@fedoraproject.org 2009-06-15 22:35:19 EDT --- perl-SystemPerl-1.321-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
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=478759
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|1.321-1.fc11 |1.321-1.fc10
package-review@lists.fedoraproject.org