Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 9:31 AM, Terry Brown wrote: > > I noticed spell check writing to a file in the leo install dir > (plugins) yesterday also. When you add a word - not sure if it's added > elsewhere as well. Please add a bug about this matter. Local dictionaries probably should be written to the home/.leo folder. Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, 24 Jun 2009 08:43:05 -0500 "Edward K. Ream" wrote: > I just merged your code. I approve of what you have done, namely > making home/.leo the default place for writing this file. I'll > recommit after adding a comment to the disabled setting node > explaining the situation. I noticed spell check writing to a file in the leo install dir (plugins) yesterday also. When you add a word - not sure if it's added elsewhere as well. Cheers -Terry --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 8:43 AM, Edward K. Ream wrote: > > > Am I correct in assuming the code still supports the setting? >> >> Yes you are. I only changed the path used when the setting is not >> available. > > > I just merged your code. I approve of what you have done, namely making > home/.leo the default place for writing this file. I'll recommit after > adding a comment to the disabled setting node explaining the situation. > Done at the trunk at rev 2087. Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 8:16 AM, Ville M. Vainio wrote: > > > Am I correct in assuming the code still supports the setting? > > Yes you are. I only changed the path used when the setting is not > available. I just merged your code. I approve of what you have done, namely making home/.leo the default place for writing this file. I'll recommit after adding a comment to the disabled setting node explaining the situation. Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 4:14 PM, Edward K. Ream wrote: >> The setting is still there, I just prepended the @ to disable it. > > Am I correct in assuming the code still supports the setting? Yes you are. I only changed the path used when the setting is not available. -- Ville M. Vainio http://tinyurl.com/vainio --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 8:10 AM, Ville M. Vainio wrote: > > On Wed, Jun 24, 2009 at 4:08 PM, Edward K. Ream > wrote: > > >> I disabled the following setting in leoSettings.leo: > >> > >> @@string script_file_path = ../test/scriptFile.py > > > Please restore the setting. It's fine to change the default location to > the > > .leo directory. > > The setting is still there, I just prepended the @ to disable it. Am I correct in assuming the code still supports the setting? Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 4:08 PM, Edward K. Ream wrote: >> I disabled the following setting in leoSettings.leo: >> >> @@string script_file_path = ../test/scriptFile.py > Please restore the setting. It's fine to change the default location to the > .leo directory. The setting is still there, I just prepended the @ to disable it. -- Ville M. Vainio http://tinyurl.com/vainio --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
Re: scriptFile.py warning
On Wed, Jun 24, 2009 at 3:10 AM, Ville M. Vainio wrote: > > I disabled the following setting in leoSettings.leo: > > @@string script_file_path = ../test/scriptFile.py > > I also changed the default script file to ~/.leo/scriptFile.py. > > This was required to make the ctrl+b work with the debian package. > > It's a bad idea to write relative to leo load dir, because it's a read > only directory if leo is "installed" (debian package, setup.py) as > opposed to just being run locally. True. > > I guess this is one of the @settings that should probably be removed > altogether; if a user wants to break his leo installation, he should > at least be required to edit the source code to do that ;-) No. One should only need to change the default of the setting. There is nothing improper about letting users choose where to write files. The explanation of the setting (in the body text of the @string node) can discuss the pros and cons. Please restore the setting. It's fine to change the default location to the .leo directory. Edward --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---
scriptFile.py warning
I disabled the following setting in leoSettings.leo: @@string script_file_path = ../test/scriptFile.py I also change the default script file to ~/.leo/scriptFile.py. This was required to make the ctrl+b work with the debian package. It's a bad idea to write relative to leo load dir, because it's a read only directory if leo is "installed" (debian package, setup.py) as opposed to just being run locally. I guess this is one of the @settings that should probably be removed altogether; if a user wants to break his leo installation, he should at least be required to edit the source code to do that ;-) -- Ville M. Vainio http://tinyurl.com/vainio --~--~-~--~~~---~--~~ You received this message because you are subscribed to the Google Groups "leo-editor" group. To post to this group, send email to leo-editor@googlegroups.com To unsubscribe from this group, send email to leo-editor+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/leo-editor?hl=en -~--~~~~--~~--~--~---