[Frugalware-devel] Re: setup repo

[Cláudio Henrique]

yeah... when this e-mail was sent I haven't read the fw_system function yet.

I'll think about the umount and rm funcs when I get home (still at work).

btw, does anyone know any good (and free) c/c++ editor for windows?
unfortunetely, I must use windows at work...

thanks for the replies.

ps.: I was really sugesting using umount(). :o)
it is used at eject but not at umount_if_needed.


On 3/27/07, VMiklos <vmiklos at frugalware.org> wrote:
> second private mail ;)
>
> Na Tue, Mar 27, 2007 at 02:45:40PM -0300, Cláudio Henrique <claudiohfg at gmail.com> pisal(a):
> > still in util.c, why not use
>
> > system(g_strdup_printf("rm -rf %s", path));
>
> > instead of
>
> we have that function in util.c, so feel free to send a patch that
> changes this, you have reason. not speaking about the above line causes
> a memleak, the proper code would be:
>
> char *ptr = g_strdup_printf("command %s", path);
> system(ptr);
> free(ptr);
>
> if you find other similar problems elsewhere (which i likely), then
> please send a patch to correct those problems
>
> thanks,
> VMiklos
>
> --
> developer of Frugalware Linux - http://frugalware.org
>
>


-- 
In order to be effective truth must penetrate like an arrow - and that
is likely to hurt. 'Posthumous Pieces' by Wei Wu Wei