<div dir="ltr">Excellent! Thanks.<div><br><div>Martin</div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 11 June 2017 at 01:22, DM Smith <span dir="ltr"><<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Fixed in SwordBookDriver.<wbr>registerNewBook(sbmd) by loading the newly downloaded conf into a new sbmd and using that to construct the book loaded into the installed book list.<span class="HOEnZb"><font color="#888888"><div><br></div><div>DM Smith</div></font></span><div><div class="h5"><div><br><div><blockquote type="cite"><div>On Jun 10, 2017, at 4:00 PM, DM Smith <<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>> wrote:</div><br class="m_7381048526863766242Apple-interchange-newline"><div><div style="word-wrap:break-word">The problem is in AbstractSwordInstaller lines 259-262:<div><div style="margin:0px;font-size:11px;line-height:normal;font-family:Monaco;color:rgb(78,144,114)"><span> </span>// TRANSLATOR: Progress label for installing the conf file for a book.</div><div style="margin:0px;font-size:11px;line-height:normal;font-family:Monaco"> <span style="color:#7e504f">job</span>.setSectionName(JSMsg.<wbr>gettext(<span style="color:#3933ff">"Copying config file"</span>));</div><div style="margin:0px;font-size:11px;line-height:normal;font-family:Monaco"> <span style="color:#7e504f">sbmd</span>.setLibrary(NetUtil.<wbr>getURI(<span style="color:#7e504f">dldir</span>));</div><div style="margin:0px;font-size:11px;line-height:normal;font-family:Monaco"> SwordBookDriver.<wbr>registerNewBook(<span style="color:#7e504f">sbmd</span>);</div><div><br></div><div>It actually doesn’t attempt to do what it says. The config file has been copied. The sbmd.setLibrary call is an attempt to take the mods.d.tar.gz entry and massage it enough to make it work. It isn’t sufficient. Still trying to figure out what should be done here.</div><div><br></div><div>So, either we can load the new conf or massage the other enough to make it work.</div><div><br></div><div>I’m thinking that it’d be safer to load it anew. If I figure out the n things that it needs that loading from file provides, who is to say that tomorrow loading won’t change and break it again.</div><div><br></div><div>In Him,</div><div><span class="m_7381048526863766242Apple-tab-span" style="white-space:pre-wrap"> </span>DM</div><div><br></div><div><blockquote type="cite"><div>On May 29, 2017, at 5:13 PM, Martin Denham <<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>> wrote:</div><br class="m_7381048526863766242Apple-interchange-newline"><div><div dir="ltr">.. or maybe you could derive the real conf name exactly from attributes in the partial sbmd, without iterating over mods.d, and load the correct conf immediately</div><div class="gmail_extra"><br><div class="gmail_quote">On 29 May 2017 at 21:49, Martin Denham <span dir="ltr"><<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks DM, I could take a stab at a fix but I have not got a great understanding of JSword at this level.<div><br></div><div>It seems like, somewhere, we should </div><div><ol><li>iterate over the mods.d folder and find the real .conf file matching the incomplete sbmd by matching initials or abbreviation<br></li><li>generate a real sbmd from the real conf<br></li><li>either just return the real sbmd or if part of install() then registerNewBook(realSbmd)</li></ol></div><div>Enjoy your holiday.<span class="m_7381048526863766242HOEnZb"><font color="#888888"><br></font></span></div><span class="m_7381048526863766242HOEnZb"><font color="#888888"><div><br></div><div>Martin</div></font></span></div><div class="m_7381048526863766242HOEnZb"><div class="m_7381048526863766242h5"><div class="gmail_extra"><br><div class="gmail_quote">On 29 May 2017 at 19:09, DM Smith <span dir="ltr"><<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>I'm on holiday. Will look at it soon after. <span class="m_7381048526863766242m_9056161251179281105HOEnZb"><font color="#888888"><br><br>DM </font></span></div><div><div class="m_7381048526863766242m_9056161251179281105h5"><div><br>On May 29, 2017, at 9:40 AM, Martin Denham <<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">I got BibleDesktop running locally and it has the same problem.<div><br></div><div>1. Run BibleDesktop</div><div>2. Install a new module e.g. Darby</div><div>3. View 'Installed Books'</div><div>4. Select the new book e.g. Darby</div><div>5. Cannot delete because the 'Delete Book' button is disabled due to the same issue as above</div><div><br></div><div>Martin<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 28 May 2017 at 15:20, Martin Denham <span dir="ltr"><<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Yes, are you aware of any way to tell JSword to load the real conf file rather than the buffer extracted from mods.d.tar.gz? I have looked around but can't see any way.<span class="m_7381048526863766242m_9056161251179281105m_3026103921434705269HOEnZb"><font color="#888888"><div><br></div><div>Martin</div></font></span></div><div class="m_7381048526863766242m_9056161251179281105m_3026103921434705269HOEnZb"><div class="m_7381048526863766242m_9056161251179281105m_3026103921434705269h5"><div class="gmail_extra"><br><div class="gmail_quote">On 28 May 2017 at 03:59, DM Smith <span dir="ltr"><<a href="mailto:dmsmith@crosswire.org" target="_blank">dmsmith@crosswire.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Sorry for the late response.<div><br></div><div>While we have moved JSword to git, BibleDesktop is still in SVN. You can find the info on this page: <a href="http://www.crosswire.org/jsword/svn.html" target="_blank">http://www.crosswire.org<wbr>/jsword/svn.html</a> The page hasn’t been updated since we moved JSword to git, but it is still accurate wrt BibleDesktop.<div><br></div><div>Looking briefly the basic issue is that when we load a conf from mods.d.tar.gz we are loading a buffer not reading the file from disk.</div><div><br></div><div>Hope this helps,</div><div>DM Smith</div><div><br><div><blockquote type="cite"><div><div class="m_7381048526863766242m_9056161251179281105m_3026103921434705269m_-8070710422050328974h5"><div>On May 27, 2017, at 6:36 PM, Martin Denham <<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>> wrote:</div><br class="m_7381048526863766242m_9056161251179281105m_3026103921434705269m_-8070710422050328974m_1098445200005498684Apple-interchange-newline"></div></div><div><div><div class="m_7381048526863766242m_9056161251179281105m_3026103921434705269m_-8070710422050328974h5"><div dir="ltr">I was actually hoping to dive into BibleDesktop in the hope of finding a solution to <a href="http://tracker.crosswire.org/projects/JS/issues/JS-282" target="_blank">JS-282</a>. I saw some BibleDesktop source in a couple of places but neither looked very official, so that is why I asked.<div><br></div><div>However, if anybody can suggest a possible solution for <a href="http://tracker.crosswire.org/projects/JS/issues/JS-282" target="_blank">JS-282</a> I would be very thankful and would not need to analyse BD.</div><div><br></div><div>Martin</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 22 April 2017 at 23:32, Martin Denham <span dir="ltr"><<a href="mailto:mjdenham@gmail.com" target="_blank">mjdenham@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Where is the latest BibleDesktop source stored?<span class="m_7381048526863766242m_9056161251179281105m_3026103921434705269m_-8070710422050328974m_1098445200005498684HOEnZb"><font color="#888888"><div><br></div><div>Martin</div></font></span></div>
</blockquote></div><br></div></div></div>
______________________________<wbr>_________________<br>jsword-devel mailing list<br><a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" target="_blank">http://www.crosswire.org/mailm<wbr>an/listinfo/jsword-devel</a><br></div></blockquote></div><br></div></div></div><br>______________________________<wbr>_________________<br>
jsword-devel mailing list<br>
<a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/mailm<wbr>an/listinfo/jsword-devel</a><br>
<br></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></blockquote><blockquote type="cite"><div><span>______________________________<wbr>_________________</span><br><span>jsword-devel mailing list</span><br><span><a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a></span><br><span><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" target="_blank">http://www.crosswire.org/mailm<wbr>an/listinfo/jsword-devel</a></span><br></div></blockquote></div></div></div><br>______________________________<wbr>_________________<br>
jsword-devel mailing list<br>
<a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/mailm<wbr>an/listinfo/jsword-devel</a><br>
<br></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
______________________________<wbr>_________________<br>jsword-devel mailing list<br><a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/jsword-devel</a><br></div></blockquote></div><br></div></div>______________________________<wbr>_________________<br>jsword-devel mailing list<br><a href="mailto:jsword-devel@crosswire.org" target="_blank">jsword-devel@crosswire.org</a><br><a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/jsword-devel</a><br></div></blockquote></div><br></div></div></div></div><br>______________________________<wbr>_________________<br>
jsword-devel mailing list<br>
<a href="mailto:jsword-devel@crosswire.org">jsword-devel@crosswire.org</a><br>
<a href="http://www.crosswire.org/mailman/listinfo/jsword-devel" rel="noreferrer" target="_blank">http://www.crosswire.org/<wbr>mailman/listinfo/jsword-devel</a><br>
<br></blockquote></div><br></div>