Le 19/02/2015 00:53, Fernando de Oliveira a écrit : > On 18-02-2015 18:39, Pierre Labastie wrote: >> Le 18/02/2015 12:29, Pierre Labastie a écrit : >>> Le 18/02/2015 12:00, Fernando de Oliveira a écrit : >>>> On 17-02-2015 13:53, Fernando de Oliveira wrote: >>>>> Commit was pushed to LO-4-4 and LO-4-3. I'm at this moment running a >>>> Fixed at revision 15522, for i686. >>>> >>>> I understand that Pierre is taking care of x86_64 case: whether or not >>>> is needed, or if it doesn't matter if used or not? >>>> >>> Sending the same message,as in the other (OT) thread: >>> Right now, building LO without the patch on a 64 bit VM. Will let you know >>> if >>> it passes. >> >> OK, LO-4.4 builds fine (and seems to work) without the patch on 64 bit. > > Good. But the next version will come with it, probably, unless they fix > what is wrong or include an "if". > >> I have a little problem with the instructions, though. At the end, there is >> the command: >> -------- >> update-desktop-database >> -------- >> which is not present on my system, so that the automatically generated script >> fails at this point. > > Sorry about that. > >> This utility is from the desktop-file-utils package, which is not even among >> the optional deps of LO. So, what should we do? > > >> Remove this instruction? > > Please, don't. > >> tag desktop-file-utils as recommended? > >> tag it as optional, and add an entry in >> command explanations? > > It clearly needs desktop file utils even if the instruction is removed, > because the desktop files need to be installed. But some people use just > windows managers, not DE, thus, these files are useless for them. it > should be recommended, if building for a desktop environment, but > optional if not. > > I would put in recommended with "(for update-desktop-database - see > Command Explanations)" > > In Command Explanation: > > update-desktop-database: this ... for DFU .. > > I really don't know how to explain. Surely it is recommended for desktop > environments, almost useless, otherwise. > > Perhaps move the command in general (/usr or /opt/libre...) to > Configuration and explain there when it is needed. >
I do not know what update-desktop-database is supposed to do, so I trust you, Fernando. OTOH, I have a DE (KDE), I haven't run that command, and libreoffice appears in the application menu, so what should I expect more? (I want to understand if I am to explain something in the book). Maybe it is useful only for other desktops (lxde, xfce). I have not tested those. Let me suggest the following: - add DFU as optional dep. - separate the "update-deskto-database" instruction, and add a sentence explaining what it does, and that you need DFU. I think it is not really "configuration" (that is, something about the way the package runs), but rather "installation" (that is, something that is needed for the package to run smoothly). Pierre -- http://lists.linuxfromscratch.org/listinfo/blfs-dev FAQ: http://www.linuxfromscratch.org/blfs/faq.html Unsubscribe: See the above information page
