Re: [E-devel] imlib2 xpm buffer overflow

2005-01-06 Thread Vincent Torri


On Tue, 4 Jan 2005, Carsten Haitzler wrote:

> On Sun, 12 Dec 2004 23:41:16 -0500 "Laurence J. Lane" <[EMAIL PROTECTED]>
> babbled:
>
> > The xpm buffer overflow reported for imlib also appears to affect
> > imlib2. You can read the details at: http://bugs.debian.org/285138
>
> (a bit slow thanks to xmas and new years...) but.. fixed in cvs.  i am tempted
> to release imlib2 1.2.0 - does anyone have any objections, patches etc. etc.
> etc. ?

no objection, just the patch i've send you for the hsv<->rgb conversion

Vincent Torri


---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Michael Jennings
On Thursday, 06 January 2005, at 14:44:10 (-0500),
Peter Hyman wrote:

> Precisely the point. Because no group is set, a default is assigned.
> That is wrong.

If you're root, yes.  As a user, no.  In any event, proper packaging
results in this not mattering one wit because the packaging
instructions (like the ones in the spec file) determine user and group
ownerships.

And in any event, group ownership doesn't matter.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "So tonight I'll ask the stars above, how did I ever win your love?
  What did I do?  What did I say to turn your Angel Eyes my way?"
  -- Jeff Healy Band, "Angel Eyes"


---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Michael Jennings
On Thursday, 06 January 2005, at 14:29:18 (-0500),
Peter Hyman wrote:

> On my system, Group becomes a 601 for some reason which in packaging
> is unacceptable. Either set the group too or omit this.

[EMAIL PROTECTED] ~/cvs >> grep -ri group e16/e/themes/Makefile*
[EMAIL PROTECTED] ~/cvs >> 

'Nuff said.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "That little Huggie bastard just threw a spear at me!"
   -- Calista Flockhart, "Ally McBeal"


---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Peter Hyman
On my system, Group becomes a 601 for some reason which in packaging is
unacceptable. Either set the group too or omit this.

On Thu, 2005-01-06 at 14:22 -0500, Michael Jennings wrote:
> On Thursday, 06 January 2005, at 12:11:42 (-0500),
> Peter Hyman wrote:
> 
> > make install in e/themes is perhaps incorrect in that it tries and
> > assign a group equal to the themes install directory.
> 
> Not on this planet, it doesn't.  Where on earth did you come up with
> that idea?
> 
> > install-data-local:
> > $(mkinstalldirs) $(THEME_DIR)
> > for tf in $(THEME_FILES); do \
> > gzip -d -c < $(srcdir)/$$tf | (cd $(THEME_DIR); tar -xmf -); \
> > done
> > -if [ -n "$(USER)" ]; then chown -R $(USER) $(THEME_DIR); else chown -R 
> > root $(THEME_DIR); fi
> > 
> > Did you mean maybe $(GROUP)? Why is this even here?
> 
> Umm, no.  chOWN is not the same as chGRP.  Furthermore, the command
> cannot cause the build to fail; it is preceeded by a -.
> 
> The command is correct.
> 
> Michael
> 
-- 
Peter



---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Michael Jennings
On Thursday, 06 January 2005, at 12:11:42 (-0500),
Peter Hyman wrote:

> make install in e/themes is perhaps incorrect in that it tries and
> assign a group equal to the themes install directory.

Not on this planet, it doesn't.  Where on earth did you come up with
that idea?

> install-data-local:
>   $(mkinstalldirs) $(THEME_DIR)
>   for tf in $(THEME_FILES); do \
>   gzip -d -c < $(srcdir)/$$tf | (cd $(THEME_DIR); tar -xmf -); \
>   done
>   -if [ -n "$(USER)" ]; then chown -R $(USER) $(THEME_DIR); else chown -R 
> root $(THEME_DIR); fi
> 
> Did you mean maybe $(GROUP)? Why is this even here?

Umm, no.  chOWN is not the same as chGRP.  Furthermore, the command
cannot cause the build to fail; it is preceeded by a -.

The command is correct.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "I'll be leaving soon; it's hard to say when I'll return, and I don't
  want to lead you on.  So if you feel the need, close your eyes and
  share this dream.  It will be Eternity."
  -- Blessid Union of Souls, "Forever for Tonight"


---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Peter Hyman
On Thu, 2005-01-06 at 14:34 -0500, Michael Jennings wrote:
> On Thursday, 06 January 2005, at 14:29:18 (-0500),
> Peter Hyman wrote:
> 
> > On my system, Group becomes a 601 for some reason which in packaging
> > is unacceptable. Either set the group too or omit this.
> 
> [EMAIL PROTECTED] ~/cvs >> grep -ri group e16/e/themes/Makefile*
> [EMAIL PROTECTED] ~/cvs >> 

Precisely the point. Because no group is set, a default is assigned.
That is wrong. As Kim pointed out, this method is odd to say the least.

> 
> 'Nuff said.

Michael, not everyone knows as much as you do, but you do not need to be
condescending to those trying to learn and help.

> 
> Michael
> 
-- 
Peter



---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] e16.8 in CVS

2005-01-06 Thread Kim Woelders
cypher wrote:
It's great to see that E has begun to make use of the new X 
extensions before the other window managers. :)

Are there any plans to use the new composite manager in conjunction 
with the pager and/or iconified windows? It would be awfully nice 
(and very useful) to have off-screen window-contents updated in 
real-time on the pager's display and on iconified window 
representations.

Most of the the code to do that is already in there, at least for the
iconboxes, but has been disabled for the moment. The only thing missing
is a regular refresh of the window snapshots, which can be done in more
and less clever ways. It should be fairly easy to extend this to the
pagers, I think. Some day... :)
/Kim
---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Kim Woelders
See thread:
http://sourceforge.net/mailarchive/message.php?msg_id=6022045.
I have regularly considered expanding the *.etheme's into plain CVS
trees. That should bypass the problem entirely.
IMO, archiving the .etheme's as such goes against the whole purpose of
CVS anyway.
/Kim
Peter Hyman wrote:
make install in e/themes is perhaps incorrect in that it tries and
assign a group equal to the themes install directory. This, of course,
cannot work.
e/themes/Makefile.am
install-data-local:
$(mkinstalldirs) $(THEME_DIR)
for tf in $(THEME_FILES); do \
gzip -d -c < $(srcdir)/$$tf | (cd $(THEME_DIR); tar -xmf -); \
done
-if [ -n "$(USER)" ]; then chown -R $(USER) $(THEME_DIR); else chown -R 
root $(THEME_DIR); fi
Did you mean maybe $(GROUP)? Why is this even here?

---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] Odd chown assignment in e16.8.0 themes

2005-01-06 Thread Peter Hyman
make install in e/themes is perhaps incorrect in that it tries and
assign a group equal to the themes install directory. This, of course,
cannot work.

e/themes/Makefile.am

install-data-local:
$(mkinstalldirs) $(THEME_DIR)
for tf in $(THEME_FILES); do \
gzip -d -c < $(srcdir)/$$tf | (cd $(THEME_DIR); tar -xmf -); \
done
-if [ -n "$(USER)" ]; then chown -R $(USER) $(THEME_DIR); else chown -R 
root $(THEME_DIR); fi

Did you mean maybe $(GROUP)? Why is this even here?
-- 
Peter



---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


Re: [E-devel] Spec file(s) patche

2005-01-06 Thread Michael Jennings
On Thursday, 06 January 2005, at 09:01:31 (-0500),
Jason Williams wrote:

> I was messing around with the CVS code last night and got all the
> stuff to build for e17/apps/e.  Attached is a spec file for the libs
> and such so that you can easily build the RPMS for them.  The only
> thing that probably should be committed is the change for freetype
> in the evas package.  On Fedora, the package is freetype-2.x.x on
> other boxes it might be noted as freetype2...  just a inconsistancy
> I came across anyway, enjoy.

Applied, thanks.

Michael

-- 
Michael Jennings (a.k.a. KainX)  http://www.kainx.org/  <[EMAIL PROTECTED]>
n + 1, Inc., http://www.nplus1.net/   Author, Eterm (www.eterm.org)
---
 "The first lesson reading teaches is how to be alone."
   -- Jonathan Franzen


---
The SF.Net email is sponsored by: Beat the post-holiday blues
Get a FREE limited edition SourceForge.net t-shirt from ThinkGeek.
It's fun and FREE -- well, almosthttp://www.thinkgeek.com/sfshirt
___
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


[E-devel] Doxygen small start

2005-01-06 Thread till varoquaux
I decided I might give it a try at commenting e 17. This about the first 
time I deal with C and I have never used doxygen before.
I'd like someone to review my patch and tell me if it can be of any use 
(or if I'm just wasting my time). Comments and constructive feedback are 
welcomed. If my doxygen comments are dumb and useless please let me know 
I won't be offended.
Cheers,
Till
diff -Naur src.old/bin/e_module.h src/bin/e_module.h
--- src.old/bin/e_module.h  2005-01-06 03:43:49.833807960 +0100
+++ src/bin/e_module.h  2005-01-06 16:21:02.984680008 +0100
@@ -17,10 +17,42 @@
void*handle;

struct {
+  /**
+  @brief This is called when the module is initialised. 
+  
+  Use it to fill the menu and whatever else you should do to make the 
+  module availlable to the user. However this souldn't start the module
+  without any checks. 
+  \n Since the module has it's own menu it is best to let the user start 
+  it from there (and choose wether he wants or not to have the module 
+  running).
+  */
   void * (*init)(E_Module *m);
+  /**
+  @brief Exits the module
+
+  Stops the modules and cleans up behind (removes the entries in the 
+  contextual menu...)
+  */
   int(*shutdown)(E_Module *m);
+  /**
+  @brief Saves the configuration data for the modules.
+  */
   int(*save)(E_Module *m);
+  /**
+  @brief Initializes the infos used to represent this module in the menus.
+
+  Sets the icon and label associated with module.
+  [EMAIL PROTECTED] _E_Module::label},[EMAIL PROTECTED] 
_E_Module::icon_file},
+  [EMAIL PROTECTED] _E_Module::edje_icon_file} [EMAIL PROTECTED] 
_E_Module::edje_icon_key} 
+  can be set during this function (and they shouldn't be set elsewere).
+  */
   int(*info)(E_Module *m);
+  /**
+  @brief Shows info about this module.
+  
+  Usually will pop up an about dialogue box (...)
+  */
   int(*about)   (E_Module *m);
} func;

@@ -28,14 +60,48 @@

/* the module is allowed to modify these */
void*data;
+   /**
+   @brief The menu corresponding to this module (accessible via right click)
+   \n
+   It is up to the module itself to fill it (generaly during the init phase)
+   */
E_Menu  *config_menu;
 
/* modify these but only set them up when the info func is called */
/* e_module will free them when the module is freed. */
/* note you will need to malloc (strdup) these fields due to the free */
+
+   /**
+   @brief The label for this module (name that will show up in the menu)
+
+   modify these but only set it up when the info func is called 
+   e_module will free them when the module is freed.
+   \warning note you will need to malloc (strdup) this field due to the free 
+   */
char*label;
+  /**
+   @brief The icon for this module (will show up in the right click desktop 
+   menu)
+
+   modify this but only set it up when the info func is called 
+   \n
+   e_module will free it when the module is freed.
+   \warning note you will need to malloc (strdup) this field due to the free 
+   */
char*icon_file;
+  /**
+   modify this but only set it up when the info func is called 
+   \n
+   e_module will free it when the module is freed.
+   \warning note you will need to malloc (strdup) this field due to the free
+   */
char*edje_icon_file;
+  /**
+   modify this but only set it up when the info func is called 
+   \n
+   e_module will free it when the module is freed.
+   \warning note you will need to malloc (strdup) this field due to the free
+   */
char*edje_icon_key;
 };
 
diff -Naur src.old/modules/test/e_mod_main.c src/modules/test/e_mod_main.c
--- src.old/modules/test/e_mod_main.c   2005-01-06 03:43:49.849805464 +0100
+++ src/modules/test/e_mod_main.c   2005-01-06 04:09:26.096113648 +0100
@@ -1,6 +1,16 @@
+/**
+ @file test/e_mod_main.c
+ @brief A sample minimale module
+ */
+
 #include "e.h"
 #include "e_mod_main.h"
 
+/**
+ @brief Initializes the module
+ The options are added in the module menu (right click). However the module 
+ itself shouldn't be lauched.
+*/
 void *
 init(E_Module *m)
 {


[E-devel] Spec file(s) patche

2005-01-06 Thread Jason Williams
I was messing around with the CVS code last night and got all the stuff to
build for e17/apps/e.  Attached is a spec file for the libs and such so
that you can easily build the RPMS for them.  The only thing that probably
should be committed is the change for freetype in the evas package.  On
Fedora, the package is freetype-2.x.x on other boxes it might be noted as
freetype2...  just a inconsistancy I came across anyway, enjoy.



-- 
Jason Williams
Lead Developer
Gaim Assistant
<[EMAIL PROTECTED]>diff -ur e17/apps/e/enlightenment.spec e17-vanilla/apps/e/enlightenment.spec
--- e17/apps/e/enlightenment.spec	2005-01-05 00:05:23.0 -0500
+++ e17-vanilla/apps/e/enlightenment.spec	2005-01-05 19:31:02.802239769 -0500
@@ -18,6 +18,16 @@
 %description
 Enlightenment is a window manager.
 
+%package devel
+Summary: Development headers for Enlightenment. 
+Group: User Interface/Desktops
+Requires: %{name} = %{version}
+Requires: libjpeg-devel XFree86-devel eet-devel embryo-devel
+Requires: evas-devel edb-devel edje-devel imlib2-devel ecore-devel
+
+%description devel
+Development headers for Enlightenment.
+
 %prep
 %setup -q
 
@@ -45,4 +55,8 @@
 %{_libdir}/%{name}
 %{_datadir}/%{name}
 
+%files devel
+%defattr(-, root, root)
+%{_includedir}/enlightenment/*.h
+
 %changelog
diff -ur e17/libs/epsilon/epsilon.spec e17-vanilla/libs/epsilon/epsilon.spec
--- e17/libs/epsilon/epsilon.spec	2005-01-05 00:05:29.0 -0500
+++ e17-vanilla/libs/epsilon/epsilon.spec	2005-01-05 19:05:45.705921729 -0500
@@ -10,7 +10,6 @@
 Packager: %{?_packager:%{_packager}}%{!?_packager:Michael Jennings <[EMAIL PROTECTED]>}
 Vendor: %{?_vendorinfo:%{_vendorinfo}}%{!?_vendorinfo:The Enlightenment Project (http://www.enlightenment.org/)}
 Distribution: %{?_distribution:%{_distribution}}%{!?_distribution:%{_vendor}}
-Vendor: The Enlightenment Development Team <[EMAIL PROTECTED]>
 #BuildSuggests: xorg-x11-devel
 BuildRequires: epeg-devel imlib2-devel libjpeg-devel freetype-devel XFree86-devel
 Requires: epeg >= 0.9.0
diff -ur e17/libs/esmart/esmart.spec e17-vanilla/libs/esmart/esmart.spec
--- e17/libs/esmart/esmart.spec	2005-01-05 00:05:29.0 -0500
+++ e17-vanilla/libs/esmart/esmart.spec	2005-01-05 19:10:18.523607004 -0500
@@ -11,7 +11,6 @@
 Packager: %{?_packager:%{_packager}}%{!?_packager:Michael Jennings <[EMAIL PROTECTED]>}
 Vendor: %{?_vendorinfo:%{_vendorinfo}}%{!?_vendorinfo:The Enlightenment Project (http://www.enlightenment.org/)}
 Distribution: %{?_distribution:%{_distribution}}%{!?_distribution:%{_vendor}}
-Vendor: The Enlightenment Development Team <[EMAIL PROTECTED]>
 Requires: evas >= 1.0.0 imlib2 libjpeg ecore epsilon 
 Requires: edje freetype eet edb embryo
 #BuildSuggests: xorg-x11-devel 
diff -ur e17/libs/evas/evas.spec e17-vanilla/libs/evas/evas.spec
--- e17/libs/evas/evas.spec	2005-01-05 00:05:29.0 -0500
+++ e17-vanilla/libs/evas/evas.spec	2005-01-05 18:56:31.752387057 -0500
@@ -12,7 +12,7 @@
 Vendor: %{?_vendorinfo:%{_vendorinfo}}%{!?_vendorinfo:The Enlightenment Project (http://www.enlightenment.org/)}
 Distribution: %{?_distribution:%{_distribution}}%{!?_distribution:%{_vendor}}
 #BuildSuggests: xorg-x11-devel
-BuildRequires: libpng-devel eet-devel edb-devel XFree86-devel freetype2-devel libjpeg-devel
+BuildRequires: libpng-devel eet-devel edb-devel XFree86-devel freetype-devel libjpeg-devel
 Provides: evas evas_software_x11 evas_loader_png evas_loader_jpeg evas_loader_eet evas_loader_edb
 BuildRoot: %{_tmppath}/%{name}-%{version}-root
 
diff -ur e17/libs/ewl/ewl.spec e17-vanilla/libs/ewl/ewl.spec
--- e17/libs/ewl/ewl.spec	2005-01-05 00:05:30.0 -0500
+++ e17-vanilla/libs/ewl/ewl.spec	2005-01-05 19:16:33.186798843 -0500
@@ -72,7 +72,6 @@
 %{_bindir}/ewl-config
 %{_bindir}/ewl_test
 %{_bindir}/ewl_embed_test
-%{_bindir}/ewl_media_test
 %{_bindir}/ewl_simple_test
 
 %changelog
diff -ur e17/libs/imlib2/imlib2.spec e17-vanilla/libs/imlib2/imlib2.spec
--- e17/libs/imlib2/imlib2.spec	2005-01-04 09:47:03.0 -0500
+++ e17-vanilla/libs/imlib2/imlib2.spec	2005-01-05 18:30:01.274620690 -0500
@@ -136,12 +136,18 @@
 %defattr(-, root, root)
 %doc AUTHORS COPYING README ChangeLog doc/index.html doc/imlib2.gif doc/blank.gif
 %{_libdir}/lib*.so.*
+/usr/share/imlib2/data/fonts/*.ttf
+/usr/share/imlib2/data/images/*.png
 %{_bindir}/*
 
 %files devel
 %defattr(-, root, root, 0755)
 %{_libdir}/libImlib2.so
 %{_libdir}/*.a
+%{_libdir}/imlib2/loaders/*.a
+%{_libdir}/imlib2/loaders/*.la
+%{_libdir}/imlib2/filters/*.a
+%{_libdir}/imlib2/filters/*.la
 %{_libdir}/*.la
 %{_libdir}/pkgconfig/imlib2.pc
 %{_includedir}/*