|Summary:||No Lua Support in Celestia (i.e., for .celx scripts)|
|Product:||[Fedora] Fedora||Reporter:||Nick Urbanik <nicku>|
|Component:||celestia||Assignee:||Steven Pritchard <steve>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Fedora Extras Quality Assurance <extras-qa>|
|Fixed In Version:||celestia-1.5.0-1||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2008-03-01 15:01:47 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
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 1 Nick Urbanik 2006-07-31 01:58:52 UTC
Created attachment 133299 [details] This attachment is from http://www.archlinux.org/pipermail/tur-users/attachments/20060603/7dea8cd1/celestia-lua51.bin
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.