Bug 234842 - math/maxima: warning of missing maxima-index.lisp after latest update
Summary: math/maxima: warning of missing maxima-index.lisp after latest update
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Steve Wills
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-11 02:35 UTC by jszhao
Modified: 2019-01-13 19:24 UTC (History)
2 users (show)

See Also:
salvadore: maintainer-feedback+

Attachments
maxima - maxima-index.lisp fix (2.14 KB, patch)
2019-01-11 12:02 UTC, Lorenzo Salvadore 		salvadore: maintainer-approval+
Details | Diff
maxima - maxima-index.lisp fix (correction 1) (2.29 KB, patch)
2019-01-11 15:46 UTC, Lorenzo Salvadore 		salvadore: maintainer-approval+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description jszhao 2019-01-11 02:35:22 UTC 
The math/maxima pkg has been updated to 5.42.1. when invoking maxima, it give the following warning message:
WARNING: Maxima is unable to set up the help system.
(Details: CL-INFO::LOAD-PRIMARY-INDEX: Couldn't load
                                       "/usr/local/share/info/./maxima-index.lisp":
                                       file does not exist.)

I have checked the /usr/local/share/info/, there is maxima.info, but no maxima-index.lisp
Comment 1 Lorenzo Salvadore freebsd_committer freebsd_triage 2019-01-11 09:56:38 UTC 
I'm working on it. I'll fix it as soon as possible.
Comment 2 Lorenzo Salvadore freebsd_committer freebsd_triage 2019-01-11 12:02:50 UTC 
Created attachment 201019 [details]
maxima - maxima-index.lisp fix

Here is the fix.

The file was installed into /usr/local/info instead of /usr/local/share/info. Now it is installed in the correct location.

I've bumped PORTREVISION.

I've also moved PLIST_SUB more consistently with the official variables ordering.

Patch has been tested successfully on my system (12.0-RELEASE amd64) and with poudriere (11.2-RELEASE amd64).
Comment 3 jszhao 2019-01-11 12:52:42 UTC 
(In reply to Lorenzo Salvadore from comment #2)

Thank you very much for the quick fix. I will try when it is available from pkg.
Comment 4 Lorenzo Salvadore freebsd_committer freebsd_triage 2019-01-11 15:29:39 UTC 
Please wait to committ the patch: something else is wrong with info files. I'm fixing it.
Comment 5 Lorenzo Salvadore freebsd_committer freebsd_triage 2019-01-11 15:46:14 UTC 
Created attachment 201033 [details]
maxima - maxima-index.lisp fix (correction 1)

Now all info files should install properly.

Because of a $ instead of $$, files maxima.info, maxima.info-1, maxima.info-2 and maxima.info-3 were not installed properly, so that help was broken. Now it's fixed.

Tested successfully on my system (12.0-RELEASE amd64) and with poudriere (11.2-RELEASE amd64).
Comment 6 commit-hook freebsd_committer freebsd_triage 2019-01-13 19:24:25 UTC 
A commit references this bug:

Author: swills
Date: Sun Jan 13 19:23:26 UTC 2019
New revision: 490170
URL: https://svnweb.freebsd.org/changeset/ports/490170

Log:
  math/maxima: fix help

  PR:		234842
  Submitted by:	Lorenzo Salvadore <phascolarctos@protonmail.ch> (maintainer)
  Reported by:	jszhao@yeah.net

Changes:
  head/math/maxima/Makefile
  head/math/maxima/pkg-plist
Comment 7 Steve Wills freebsd_committer freebsd_triage 2019-01-13 19:24:33 UTC 
Committed, thanks!