Bug 200696 - No Lua Support in Celestia (i.e., for .celx scripts)
Summary: No Lua Support in Celestia (i.e., for .celx scripts)
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: celestia
Version: 5
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Steven Pritchard
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-07-31 01:58 UTC by Nick Urbanik
Modified: 2008-03-01 15:01 UTC (History)
1 user (show)

Fixed In Version: celestia-1.5.0-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-03-01 15:01:47 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
This attachment is from http://www.archlinux.org/pipermail/tur-users/attachments/20060603/7dea8cd1/celestia-lua51.bin (2.00 KB, patch)
2006-07-31 01:58 UTC, Nick Urbanik
no flags Details | Diff
This was written with advice from Hank Ramsey. (1.02 KB, patch)
2006-07-31 02:02 UTC, Nick Urbanik
no flags Details | Diff
The new spec file that applies the patches and adds the --with-lua option to the configure script. (7.12 KB, text/plain)
2006-07-31 02:05 UTC, Nick Urbanik
no flags Details

Description Nick Urbanik 2006-07-31 01:58:49 UTC
Description of problem: Celestia has not been compiled with support for lua.


Version-Release number of selected component (if applicable): 1.4.1-2.fc5


How reproducible: Always


Steps to Reproduce:
1. Open Celestia
2. Select File->Open Script
3. Be unable to run any .celx scripts
  
Actual results: Cannot run any .celx scripts


Expected results: See the .celx scripts appear in file browser, be able to
execute them successfully.


Additional info: After applying the two patches attached, and modifying the spec
file appropriately, it works.  Please apply the fix, since, .celx files provide
proper scripting, whereas the legacy .cel files are much more limited.

Comment 2 Nick Urbanik 2006-07-31 02:02:19 UTC
Created attachment 133300 [details]
This was written with advice from Hank Ramsey.

Without this patch, Celestia segfaults when try to load a .celx file.  This was
discussed on the celestia-developers list, with Hank Ramsey.

Comment 3 Nick Urbanik 2006-07-31 02:05:57 UTC
Created attachment 133301 [details]
The new spec file that applies the patches and adds the --with-lua option to the configure script.

This is the spec file I modified to successfully create the packages at
http://nicku.org/software/celestia/

I also added the developer documentation, as mentioned in the changelog.

Comment 4 Steven Pritchard 2006-07-31 16:51:54 UTC
We can't mark celestia.cfg as %config as-is.  It will need to move to /etc.

Otherwise, I've incorporated everything in 1.4.1-3 (building in devel now, and
I'll build it for FC5 if that goes well).

Thanks!

Comment 5 Steven Pritchard 2006-08-08 18:50:05 UTC
I had to add a couple of BuildRequires to build in devel, but that's done now.  
I'm building for FC-5 now.

I'll leave this open until we figure out what to do about the celestia.cfg 
issue.

Comment 6 Marek Mahut 2008-03-01 15:01:47 UTC
Lua support is build in. Regarding the celestia.cfg configuration file, I think
we should not keep it under /etc as it's not ordinary (per user) configuration
file. 

I'm closing this bug, if you think that the configuration file should go
elsewhere, please open a new bug report.


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