[Nice] R: R: R: patches for gsocket support and gtk-doc missing

[Della Betta Filippo]

Here is a patch that should fix this issue (I hope so at least:) )
I created a blank file named gtk-doc.make in autogen.sh
If gtkdocize exists, the empty gtk-doc.make is replaced from the output of gtkdocize, otherwise a blank gtk-doc.make is harmless for Makefile.am
What do you think ?
Regards,
Filippo

-----Messaggio originale-----
Da: nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedesktop.org [mailto:nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedesktop.org] Per conto di Della Betta Filippo
Inviato: mercoledì 7 marzo 2012 10:07
A: 'Youness Alaoui'; 'nice at lists.freedesktop.org'
Oggetto: [Nice] R: R: patches for gsocket support and gtk-doc missing

You're right.
I have to think to another method to exclude that include..
Regards,
Filippo

-----Messaggio originale-----
Da: nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedesktop.org [mailto:nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedesktop.org] Per conto di Youness Alaoui
Inviato: martedì 6 marzo 2012 21:50
A: nice at lists.freedesktop.org
Oggetto: Re: [Nice] R: patches for gsocket support and gtk-doc missing

Yeah, that's what I thought, but I think the '-' before the include only works for the Makefile, but gtk-doc.make is not a Makefile file, it's a Makefile.am file...
What happens is that I did a ./autogen.sh, then ./configure, then look at the Makefile that gets generates by automake, it will have the "-include gtk-doc.make" in it, and when you run make, it will complain that the file is missing a separator, etc...
The problem is that the gtk-doc.make must be parsed by automake and must be used to generate the Makefile, but with your patch, it wasn't included by the Makefile.am, and automake ignored it and it would then be included by the Makefile itself when you type 'make', which makes it crash because it's not a proper makefile file format.

On 03/06/2012 03:53 AM, Della Betta Filippo wrote:
> The '-' before the include means that if you don't find the file, the process goes on and doesn't stop with an error.
> This happens when you don't call gtkdocize (since you don't have 
> gtk-doc installed) before autoreconf -fi The patch must be applied to Makefile.am since Makefile is generated from that by automake.
> Which error did you face ?
> Regards,
> Filippo
> 
> -----Messaggio originale-----
> Da: 
> nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedesktop.org
> [mailto:nice-bounces+filippo.dellabetta=telecomitalia.it at lists.freedes
> ktop.org] Per conto di Youness Alaoui
> Inviato: martedì 6 marzo 2012 02:50
> A: nice at lists.freedesktop.org
> Oggetto: Re: [Nice] patches for gsocket support and gtk-doc missing
> 
> I merged both but I had to modify the second one, this chunk broke the makefile though :
> 
> -include $(top_srcdir)/gtk-doc.make
> +-include $(top_srcdir)/gtk-doc.make
> 
> I don't know why you added that '-' in front of the include. It made it include the gtk-doc.make file from the Makefile not from the Makefile.am, which caused it to error out.
> 
> On 03/05/2012 06:36 AM, Della Betta Filippo wrote:
>> Hi,
>>
>> Attached you can find patches for
>>
>> 1)      Support for msys/mingw and Visual Studio for gsocket (added gio
>> dependency), after Livio's patches
>>
>> 2)      Support for msys/mingw when gtk-doc is not installed
>>
>> Regards,
>>
>> Filippo Della Betta
>>
>>
>>
>> Questo messaggio e i suoi allegati sono indirizzati esclusivamente 
>> alle persone indicate. La diffusione, copia o qualsiasi altra azione 
>> derivante dalla conoscenza di queste informazioni sono rigorosamente 
>> vietate. Qualora abbiate ricevuto questo documento per errore siete 
>> cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.
>>
>> /This e-mail and any attachments// is //confidential and may contain 
>> privileged information intended for the addressee(s) only.
>> Dissemination, copying, printing or use by anybody else is 
>> unauthorised. If you are not the intended recipient, please delete 
>> this message and any attachments and advise the sender by return 
>> e-mail, Thanks./
>>
>> *rispetta l'ambienteRispetta l'ambiente. Non stampare questa mail se 
>> non è
>> necessario.*
>>
>>
>>
>> _______________________________________________
>> nice mailing list
>> nice at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/nice
> 
> 
> 
> Questo messaggio e i suoi allegati sono indirizzati esclusivamente alle persone indicate. La diffusione, copia o qualsiasi altra azione derivante dalla conoscenza di queste informazioni sono rigorosamente vietate. Qualora abbiate ricevuto questo documento per errore siete cortesemente pregati di darne immediata comunicazione al mittente e di provvedere alla sua distruzione, Grazie.
> 
> This e-mail and any attachments is confidential and may contain privileged information intended for the addressee(s) only. Dissemination, copying, printing or use by anybody else is unauthorised. If you are not the intended recipient, please delete this message and any attachments and advise the sender by return e-mail, Thanks.
> 
> _______________________________________________
> nice mailing list
> nice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/nice


_______________________________________________
nice mailing list
nice at lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/nice
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Added-support-for-compiling-when-gtk-doc-is-not-inst.patch
Type: application/octet-stream
Size: 1427 bytes
Desc: 0002-Added-support-for-compiling-when-gtk-doc-is-not-inst.patch
URL: <http://lists.freedesktop.org/archives/nice/attachments/20120307/f84258c8/attachment.obj>