Bug 235481

Summary: js.msg and jsopcode.tbl missing in devel package
Product: [Fedora] Fedora Reporter: Glen Cornell <glencoindetroit>
Component: jsAssignee: Matthias Saou <matthias>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 6   
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-04-11 21:21:08 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Glen Cornell 2007-04-06 04:03:19 UTC
Description of problem:
the following files are not found in either the js or js-devel packages:
js.msg
jsopcode.tbl
However, they are referenced in the header files:
jscntxt.h:510: #include "js.msg"
jsopcode.h:57: #include "jsopcode.tbl"

Version-Release number of selected component (if applicable):
js-devel-1.5-6.fc6

How reproducible:

Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
this appears to be a packaging defect that can be corrected by adding the two
files to the rpm spec file in the devel section.  here's a small patch:

[glen@localhost SPECS]$ diff -Naur js.spec.orig js.spec
--- js.spec.orig        2007-04-06 00:00:34.000000000 -0400
+++ js.spec     2007-04-06 00:01:16.000000000 -0400
@@ -131,6 +131,8 @@
 %{_libdir}/*.so
 %{_libdir}/*.a
 %{_includedir}/js*.h
+%{_includedir}/jsopcode.tbl
+%{_includedir}/js.msg
 
 
 %changelog

Comment 1 Glen Cornell 2007-04-06 04:46:37 UTC
Oops - please use the following patch instead.  the one above did not install
the files properly...

[glen@localhost SPECS]$ diff -Naur js.spec.orig js.spec
--- js.spec.orig        2007-04-06 00:00:34.000000000 -0400
+++ js.spec     2007-04-06 00:38:34.000000000 -0400
@@ -93,7 +93,7 @@
     %{buildroot}%{_libdir}/
 %{__mv} %{buildroot}%{_libdir}/libjs.so %{buildroot}%{_libdir}/libjs.so.1
 %{__ln_s} -nf libjs.so.1 %{buildroot}%{_libdir}/libjs.so
-%{__install} -m 0644 src/js*.h src/Linux_All_OPT.OBJ/jsautocfg.h \
+%{__install} -m 0644 src/js*.h src/js.msg src/jsopcode.tbl
src/Linux_All_OPT.OBJ/jsautocfg.h \
   %{buildroot}%{_includedir}/
 %{__install} -m 0644 libjs.pc \
     %{buildroot}%{_libdir}/pkgconfig/
@@ -131,6 +131,8 @@
 %{_libdir}/*.so
 %{_libdir}/*.a
 %{_includedir}/js*.h
+%{_includedir}/jsopcode.tbl
+%{_includedir}/js.msg
 
 
 %changelog


Comment 2 Matthias Saou 2007-04-11 21:21:08 UTC
A new package with this change has been pushed to devel and FC6. Thanks for the
report!