Bug 2744 - Essential files for Gnuplot are in gnuplot-doc package
Summary: Essential files for Gnuplot are in gnuplot-doc package
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 1
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: QA Team
QA Contact:
URL:
Whiteboard:
Keywords: validated_update
Depends on:
Blocks:
 
Reported: 2011-09-14 17:48 CEST by Jérôme Hénin
Modified: 2012-02-07 12:35 CET (History)
7 users (show)

See Also:
Source RPM: gnuplot
CVE:
Status comment:


Attachments

Description Jérôme Hénin 2011-09-14 17:48:54 CEST
The gnuplot package contains only the binary, so it is missing quite a few files that are useful to run the program. The missing files are basically the whole content of /usr/share/gnuplot, which is included in gnuplot-doc although it is not documentation!

In particular, /usr/share/gnuplot/4.4/PostScript/prologue.ps is needed for any postscript output, so I think it should be in the base package. The simplest would probably be to include all of /usr/share/gnuplot in the gnuplot package, which OpenSuse does (they are probably not the only ones).
D Morgan 2011-09-14 19:46:43 CEST

CC: (none) => dmorganec
Source RPM: (none) => gnuplot

Comment 1 Samuel Verschelde 2011-10-01 17:53:38 CEST
Assigning to maintainer

CC: (none) => stormi
Assignee: bugsquad => nanardon

Comment 2 Marja Van Waes 2012-01-07 21:57:59 CET
pinging. because nothing happened to this report since more than 3 months ago, and it still has the status NEW or REOPENED


@ Olivier

Please set status to ASSIGNED if you think this bug was assigned correctly. If for work flow reasons you can't do that, then please put OK on the whiteboard instead.

CC: (none) => marja11

Comment 3 Olivier Thauvin 2012-02-03 10:15:18 CET
I just fixed it in cooker, and submitted an update less intrusive into Mageia/1.
Comment 4 Olivier Thauvin 2012-02-03 11:19:56 CET
s/cooker/cauldron/, you fixed this yourself.

The submission has failed to Mageia/1 for some reasons.
Comment 5 Marja Van Waes 2012-02-03 11:26:34 CET
(In reply to comment #4)
> s/cooker/cauldron/, you fixed this yourself.
> 
> The submission has failed to Mageia/1 for some reasons.

shouldn't it go into updates/testing first?
if you submit there, please assign this bug to QA and put yourself in the cc of this bug
Comment 6 Olivier Thauvin 2012-02-03 12:18:20 CET
I submitted it in core/update_testing of course.

Assignee: nanardon => qa-bugs

Comment 7 claire robinson 2012-02-03 13:05:26 CET
Olivier, could you please write an update advisory so we know what you have done and can work out how to test.

Thankyou.
Comment 8 Marja Van Waes 2012-02-03 13:28:28 CET
(In reply to comment #7)
> Olivier, could you please write an update advisory so we know what you have
> done and can work out how to test.
> 
> Thankyou.

cc'ing Olivier

CC: (none) => nanardon

Comment 9 Olivier Thauvin 2012-02-06 16:08:46 CET
Really, I don't know what to say... I did not introduce the issue (It appear because someone did want to follow MDV packaging).
Is this enough ?:

"This update ensure all requirement are installed to have gnuplot working perfectly".
Comment 10 claire robinson 2012-02-06 18:38:13 CET
Olivier, for QA we have to know what to test for, so we need to know what has changed. Please refer to the updates policy on Mageia wiki for future updates to Mageia 1, it differs from Cauldron.

https://wiki.mageia.org/en/Updates_policy#Maintainer_.28or_any_interested_packager.29

Thankyou :)



Working on the assumption you have now included the postscript in the gnuplot rpm..

x86_64

Before
------

# urpme -a gnuplot
To satisfy dependencies, the following 4 packages will be removed (6.7MB):
  gnuplot-4.4.3-1.mga1.x86_64
  gnuplot-doc-4.4.3-1.mga1.x86_64
  gnuplot-mode-4.4.3-1.mga1.x86_64
  gnuplot-nox-4.4.3-1.mga1.x86_64
Remove 4 packages? (y/N) y

# rm -rf /usr/share/gnuplot

# urpmi --no-suggests gnuplot

# ls /usr/share/gnuplot/4.4/PostScript/
ls: cannot access /usr/share/gnuplot/4.4/PostScript/: No such file or directory


After
-----
# urpmi.update --no-ignore "Core Updates Testing"
# urpmi --no-suggests gnuplot
To satisfy dependencies, the following packages are going to be installed:
   Package                        Version      Release       Arch
(medium "Core Updates Testing")
  gnuplot                        4.4.3        2.mga1        x86_64
  gnuplot-doc                    4.4.3        2.mga1        x86_64
2.6MB of additional disk space will be used.
1.9MB of packages will be retrieved.
Proceed with the installation of the 2 packages? (Y/n)y

# ls /usr/share/gnuplot/4.4/PostScript/
8859-15.ps  8859-2.ps  aglfn.txt  cp1251.ps  cp850.ps  koi8r.ps  prologue.ps
8859-1.ps   8859-9.ps  cp1250.ps  cp437.ps   cp852.ps  koi8u.ps  utf-8.ps



So it appears you have changed the 'suggests' on gnuplot-doc to a hard 'require'. As gnuplot-doc will be provided in updates this won't be affected by bug 2317.

For testing I downloaded the tarball from http://downloads.sourceforge.net/project/gnuplot/gnuplot/4.4.3/gnuplot-4.4.3.tar.gz?r=http%3A%2F%2Fsourceforge.net%2Fprojects%2Fgnuplot%2Ffiles%2Fgnuplot%2F4.4.3%2F&ts=1328548522&use_mirror=freefr (direct link)

Extracted the demo directory and cd'd into it and did tests from here
http://www.gnuplot.info/screenshots/index.html

$ gnuplot

        G N U P L O T
        Version 4.4 patchlevel 3
        last modified March 2011
        System: Linux 2.6.38.8-desktop-10.mga

        Copyright (C) 1986-1993, 1998, 2004, 2007-2010
        Thomas Williams, Colin Kelley and many others

        gnuplot home:     http://www.gnuplot.info
        faq, bugs, etc:   type "help seeking-assistance"
        immediate help:   type "help"
        plot window:      hit 'h'

Terminal type set to 'wxt'
gnuplot>load "all.dem"

Pressed Return as it ran through its demos.

gnuplot> set terminal postscript color solid
Terminal type set to 'postscript'
Options are 'landscape noenhanced defaultplex \
   leveldefault color colortext \
   solid dashlength 1.0 linewidth 1.0 butt noclip \
   palfuncparam 2000,0.003 \
   "Helvetica" 14 '
gnuplot> set output "all_demos.ps"
gnuplot> load "all.dem"

Pressed Return while it did its thing again

gnuplot> quit

$ gv all_demos.ps

Shows 244 pages of graphs.


Testing complete x86_64
Comment 11 claire robinson 2012-02-06 18:39:19 CET
Suggested advisory
----------------------
If gnuplot is installed without the suggested gnuplot-doc then many features are missed as they are contained in the gnuplot-doc package.

This update adds a hard require of gnuplot-doc to gnuplot
----------------------

SRPM: gnuplot-4.4.3-2.mga1.src.rpm


Olivier, is this information correct please?
Comment 12 Olivier Thauvin 2012-02-06 18:43:59 CET
Yes, indeed, this information are corrected and seems perfect.

About cauldron: I merged the -doc package into the main one as the -doc is as the doc is very small and most of the files were need anyway.
Comment 13 claire robinson 2012-02-06 19:04:42 CET
Thankyou :)

We don't perform QA on cauldron packages, it's just Mageia 1 we need to now about.

There is a sample advisory on the policy link, it should explain it better than I can here.
Comment 14 Dave Hodgins 2012-02-06 21:17:53 CET
Testing complete on i586.  Installed from core release with --no-suggests,
gnuplot> help
/usr/share/gnuplot/4.4/gnuplot.gih: No such file or directory
Installed the update using mgaapplet.  The help now works, as
does generating a plot using copy/paste from
http://gnuplot.sourceforge.net/demo/simple.html

Could someone from the sysadmin team push the srpm
gnuplot-4.4.3-2.mga1.src.rpm
from Core Updates Testing to Core Updates.

Advisory: If gnuplot is installed without the suggested
gnuplot-doc then many features are missed, as they use
files from the gnuplot-doc package.

This update adds a hard requires of the gnuplot-doc
package to the gnuplot package.

https://bugs.mageia.org/show_bug.cgi?id=2744

Keywords: (none) => validated_update
CC: (none) => davidwhodgins, sysadmin-bugs

Comment 15 Thomas Backlund 2012-02-07 12:35:49 CET
update pushed

Status: NEW => RESOLVED
CC: (none) => tmb
Resolution: (none) => FIXED


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