Re: [Mageia-dev] php threadsafe

2012-11-12 Thread Funda Wang
2012/11/13 Thomas Spuhler 

> Since this isn't a bug, I am brining it up here:
> The 389-ds wants apache to run with the mpm_worker_module.
> Apache-2.4 provides now this module as stabel
>
> When I do this (using mpm-worker I get this error:
> Apache is running a threaded MPM, but your PHP Module is not compiled to
> be threadsafe.  You need to
> recompile PHP
> I am not quite sure how to proceed. What would be the disadvantage of
> compiling PHP threadsafe?

It is not turned on by default :(


>
> --
> Best regards
> Thomas Spuhler
>


[Mageia-dev] php threadsafe

2012-11-12 Thread Thomas Spuhler
Since this isn't a bug, I am brining it up here:
The 389-ds wants apache to run with the mpm_worker_module.
Apache-2.4 provides now this module as stabel

When I do this (using mpm-worker I get this error:
Apache is running a threaded MPM, but your PHP Module is not compiled to be 
threadsafe.  You need to 
recompile PHP
I am not quite sure how to proceed. What would be the disadvantage of compiling 
PHP threadsafe?

-- 
Best regards
Thomas Spuhler


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Il 12/11/2012 12:49, Guillaume Rousse ha scritto:
> Le 12/11/2012 10:57, Angelo Naselli a écrit :
>> -BEGIN PGP SIGNED MESSAGE- Hash: SHA1
>> 
>> Il 12/11/2012 09:44, Guillaume Rousse ha scritto:
>>> Do you use Test::Compile to check your code compiles correctly
>> Interesting, can you point me in how to use it?
> Basically, that's just a way to automatize of multiple use_ok() 
> statements, when you have lots of perl modules.
> 
> You can check module documentation on CPAN here: 
> http://search.cpan.org/~egiles/Test-Compile-0.22/
> 
> And if you want a real-world usage (with lot of project-specific
> noise), check there: 
> https://github.com/fusinv/fusioninventory-agent/blob/2.2.x/t/01compile.t
>
> 
I'll give it a look when i'm less tired...
thanks.

In the mean while i hope to have cleaned a little bit the submitted code.


Angelo
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlChb2AACgkQqEs9DA4DquBCRQCgjjaT5Noik2AHckM6J8T/M+l9
SsoAnA2GIiO3MNbS9oyJ7vU5Vf7dSwZk
=JO/Z
-END PGP SIGNATURE-


Re: [Mageia-dev] [changelog] [RPM] cauldron core/release supertux-0.3.3-39.mga3

2012-11-12 Thread Olivier Blin
Jose Jorge  writes:

> Le 12/11/2012 15:48, Guillaume Rousse a écrit :
>> Le 12/11/2012 13:16, Olivier Blin a écrit :
>>> This looks wrong, the data package does not require the main package.
>> Is it really necessary to have a cyclic dependency here ? I don't
>> imagine anyone installing 'supertux-data' alone by mistake.

Agreed, that was my point.

> Well, without this cyclic dependency, uninstalling supertux does not
> remove supertux-data. This is why I added it.

That's why we have the auto-orphans feature of urpme.
Yes, the feature might be unavailable in the GUI, but we should not add
cyclic deps just because of that.

-- 
Olivier Blin - blino


Re: [Mageia-dev] [changelog] [RPM] cauldron core/release supertux-0.3.3-39.mga3

2012-11-12 Thread Jose Jorge

Le 12/11/2012 15:48, Guillaume Rousse a écrit :

Le 12/11/2012 13:16, Olivier Blin a écrit :

This looks wrong, the data package does not require the main package.

Is it really necessary to have a cyclic dependency here ? I don't
imagine anyone installing 'supertux-data' alone by mistake.

Well, without this cyclic dependency, uninstalling supertux does not 
remove supertux-data. This is why I added it.


There is only 2 packages to be smaller for mirrors, and I dont imagine, 
I see users clicking only on softx-data to install it...


Re: [Mageia-dev] [changelog] [RPM] cauldron core/release supertux-0.3.3-39.mga3

2012-11-12 Thread Guillaume Rousse

Le 12/11/2012 13:16, Olivier Blin a écrit :

zezinho  writes:


Name: supertux Relocations: (not relocatable)


[...]


+ Revision: 317165
- make data sub package require main package


Hi,

This looks wrong, the data package does not require the main package.
Is it really necessary to have a cyclic dependency here ? I don't 
imagine anyone installing 'supertux-data' alone by mistake.


--
BOFH excuse #262:

Our POP server was kidnapped by a weasel.


Re: [Mageia-dev] NEED REVIEW: drakdisk refresh feature

2012-11-12 Thread Nicolas Lécureuil
Le lundi 12 novembre 2012 15:01:19 vous avez écrit :
> Op maandag 12 november 2012 14:15:14 schreef Nicolas Lécureuil:
> > Le lundi 12 novembre 2012 01:24:55 AL13N a écrit :
> > > Ok, so this is a new patch; this time, it's next to the toggle for
> > > expert
> > > button.
> > > 
> > > I do a new device check, in a separate variable and check with device
> > > file
> > > as unique; if there's one that isn't in the old list. (or if it's
> > > changed
> > > size). in that case i put the new entry into the old list.
> > > 
> > > however, it seems that a VM shuts itself off on executing the check when
> > > a
> > > new USB disk was added. :-(
> > > 
> > > Maybe i'm doing some fundamentals wrong, so i'm asking here if anyone
> > > has
> > > an insight, idea or direction i should go to instead.
> > > 
> > > i'm thinking because i only do 1 of the 5 preparation checks for new
> > > disks,
> > > that maybe not everything is done correctly yet: but a halting VM is not
> > > exactly what i expected...
> > > 
> > > AL13N
> > 
> > Hi,
> > 
> > just tested but nothing is done/appear when i click on refresh
> 
> well, don't forget that if you test in a VM, the host itself must detect the
> new disk too, so i always tested with adding an USB disk. Normally, you
> should see an extra tab, if it would work...

no, not in a VM, a real install.

> was this in cauldron, or mga2?
cauldron
> 
> i guess the good point is that it didn't halt either?

you can see it this way


Re: [Mageia-dev] NEED REVIEW: drakdisk refresh feature

2012-11-12 Thread AL13N
Op maandag 12 november 2012 14:15:14 schreef Nicolas Lécureuil:
> Le lundi 12 novembre 2012 01:24:55 AL13N a écrit :
> > Ok, so this is a new patch; this time, it's next to the toggle for expert
> > button.
> > 
> > I do a new device check, in a separate variable and check with device file
> > as unique; if there's one that isn't in the old list. (or if it's changed
> > size). in that case i put the new entry into the old list.
> > 
> > however, it seems that a VM shuts itself off on executing the check when a
> > new USB disk was added. :-(
> > 
> > Maybe i'm doing some fundamentals wrong, so i'm asking here if anyone has
> > an insight, idea or direction i should go to instead.
> > 
> > i'm thinking because i only do 1 of the 5 preparation checks for new
> > disks,
> > that maybe not everything is done correctly yet: but a halting VM is not
> > exactly what i expected...
> > 
> > AL13N
> 
> Hi,
> 
> just tested but nothing is done/appear when i click on refresh

well, don't forget that if you test in a VM, the host itself must detect the 
new disk too, so i always tested with adding an USB disk. Normally, you should 
see an extra tab, if it would work...

was this in cauldron, or mga2?

i guess the good point is that it didn't halt either?


Re: [Mageia-dev] [changelog] 2 tainted/updates_testing vlc-2.0.3-2.2.mga2.tainted

2012-11-12 Thread Thomas Backlund

shlomif skrev 12.11.2012 13:07:

Name: vlc  Relocations: (not relocatable)
Version : 2.0.3 Vendor: Mageia.Org
Release : 2.2.mga2.tainted  Build Date: Mon Nov 12 11:57:19 2012


[...]


shlomif  2.0.3-2.2.mga2:
+ Revision: 317171
- Bump subrel for tainted


This is wrong!

package in core and tainted should have same version-release.

BS is adapted to accept it.

just submit to one media, wait for it to be built and then submit to the 
other one...


--

Thomas




Re: [Mageia-dev] NEED REVIEW: drakdisk refresh feature

2012-11-12 Thread Nicolas Lécureuil
Le lundi 12 novembre 2012 01:24:55 AL13N a écrit :
> 
> Ok, so this is a new patch; this time, it's next to the toggle for expert
> button.
> 
> I do a new device check, in a separate variable and check with device file
> as unique; if there's one that isn't in the old list. (or if it's changed
> size). in that case i put the new entry into the old list.
> 
> however, it seems that a VM shuts itself off on executing the check when a
> new USB disk was added. :-(
> 
> Maybe i'm doing some fundamentals wrong, so i'm asking here if anyone has an
> insight, idea or direction i should go to instead.
> 
> i'm thinking because i only do 1 of the 5 preparation checks for new disks,
> that maybe not everything is done correctly yet: but a halting VM is not
> exactly what i expected...
> 
> AL13N

Hi,

just tested but nothing is done/appear when i click on refresh


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Guillaume Rousse

Le 12/11/2012 11:07, Angelo Naselli a écrit :

Again I'm not a perl developer and neither i'm pretending to be,
what i'm trying to, is porting some old programs to new layout,
but if that hurts someone's feeling i can stop working on it
and do some in c++ for what i need.
Sorry, I didn't intended to be rude here, just to give some potentially 
useful coding style hints, if you expect to avoid the recurrent 
drakxtools-perl-style-is-unmaintaible criticims. Feel free to ignore 
completly, make your own mind, whatever, I won't feel offended :)


--
BOFH excuse #38:

secretary plugged hairdryer into UPS


Re: [Mageia-dev] [changelog] [RPM] cauldron core/release supertux-0.3.3-39.mga3

2012-11-12 Thread Olivier Blin
zezinho  writes:

> Name: supertux Relocations: (not relocatable)

[...]

> + Revision: 317165
> - make data sub package require main package

Hi,

This looks wrong, the data package does not require the main package.

-- 
Olivier Blin - blino


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Il 12/11/2012 12:14, Steven Tucker ha scritto:
> Now get back and fix your rubbish code !! lol
\o/ :D
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlCg6IsACgkQqEs9DA4DquAqqACglU1H3Vs/72x5aoDBA2a92xkF
Sy0An0UyC4hTub+/3JhTgHtC9g0N2Ya1
=1OQJ
-END PGP SIGNATURE-


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Colin Guthrie
'Twas brillig, and Angelo Naselli at 12/11/12 12:09 did gyre and gimble:
>> It's very easy to use the journal APIs (either by linking the
>> library, using python bindings - not sure if there are external
>> perl bindings yet, but I wouldn't be surprised if there are - or
>> simply calling journalctl on the command line and parsing the
>> result. It offers an easy to parse JSON format so even going via
>> this route should be trivial if official bindings do not exist).
>> You can get all logs between certain dates and for particular
>> services with ease, without having to look in the right file or
>> grep for the right binary. i.e. all the building blocks are in
>> place for you :)
> I read your comment in draklog bug, i have taken a note and i will
> ask you to explain it better to me, since i'm not very familiar with
> all this stuff.
> 
> About binding, well libyui allows also to use python and ruby,
> so rewriting my rubbish - quoting Steven :p - to re-implementing
> another rubbish in python it could be possible and easy :D
> 
> Anyway, Col ehm you could help in this project...
> ...we will be very happy :D

Time is not on my side at present, but I'll do my best to advise and
help out when I can. Minimal hacking may occur but I'm afraid I can't
really promise anything at this stage.

Col

-- 

Colin Guthrie
colin(at)mageia.org
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited http://www.tribalogic.net/
Open Source:
  Mageia Contributor http://www.mageia.org/
  PulseAudio Hacker http://www.pulseaudio.org/
  Trac Hacker http://trac.edgewall.org/


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

>>> +#- check services +my ) . Data::Dumper->Dump([ $service ],
>>> [qw(*services)]) . q( +foreach (split(':', $options{SERVICES}))
>>> { +next unless $services{$_}; +$r .= "Service $_
>>> ($services{$_} is not running)\\n" unless -e 
>>> "/var/lock/subsys/$_"; +}
>> You shouldn't leave debugging statement in your code. Moreover, I
>> doubt this is syntactically correct. Do you use Test::Compile to
>> check your code compiles correctly ?
> 
> Also /var/log/subsys is almost dead under systemd anyway. It's
> still used by a few legacy initscripts that systemd calls, but I'm
> not sure where this is going.
Ehm the above code is the draklog one, and it was use to create a
wizard to setup mail alert.
It's not implemented yet, in the new log viewer so any new input can be
added now.


> Also, if you are implementing a new log viewer module, is this
> going to use the systemd journal rather than implementing it's own
> complex logic for parsing compressed text files? I've not looked at
> the code, but it would be *awesome* if this is what was happening
> :)
> 
> It's very easy to use the journal APIs (either by linking the
> library, using python bindings - not sure if there are external
> perl bindings yet, but I wouldn't be surprised if there are - or
> simply calling journalctl on the command line and parsing the
> result. It offers an easy to parse JSON format so even going via
> this route should be trivial if official bindings do not exist).
> You can get all logs between certain dates and for particular
> services with ease, without having to look in the right file or
> grep for the right binary. i.e. all the building blocks are in
> place for you :)
I read your comment in draklog bug, i have taken a note and i will
ask you to explain it better to me, since i'm not very familiar with
all this stuff.

About binding, well libyui allows also to use python and ruby,
so rewriting my rubbish - quoting Steven :p - to re-implementing
another rubbish in python it could be possible and easy :D

Anyway, Col ehm you could help in this project...
...we will be very happy :D

Cheers,
Angelo (who continues trying involve people :D)

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlCg5xAACgkQqEs9DA4DquCx1QCeN8xtJ06uUohpR0WpHzqYbbfI
xoAAn0KHEx34+MHXlr7J+hS6Jp4EGqCD
=aDG5
-END PGP SIGNATURE-


Re: [Mageia-dev] Small project for a Python programmer

2012-11-12 Thread Barry Jackson

On 11/11/12 17:35, 1 wrote:

Barry Jackson  writes:



What is needed is a lexer for rpm spec files.



https://bitbucket.org/birkenfeld/pygments-main/pull-request/124



Thanks for the heads up - good.

I guess that this will take time to filter through to a release version?



Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Guillaume Rousse

Le 12/11/2012 10:57, Angelo Naselli a écrit :

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Il 12/11/2012 09:44, Guillaume Rousse ha scritto:

Do you use Test::Compile to check your code compiles correctly

Interesting, can you point me in how to use it?
Basically, that's just a way to automatize of multiple use_ok() 
statements, when you have lots of perl modules.


You can check module documentation on CPAN here:
http://search.cpan.org/~egiles/Test-Compile-0.22/

And if you want a real-world usage (with lot of project-specific noise), 
check there:

https://github.com/fusinv/fusioninventory-agent/blob/2.2.x/t/01compile.t

--
BOFH excuse #348:

We're on Token Ring, and it looks like the token got loose.


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Guillaume Rousse

Le 12/11/2012 12:08, AL13N a écrit :

Op maandag 12 november 2012 10:57:43 schreef Angelo Naselli:

Il 12/11/2012 09:44, Guillaume Rousse ha scritto:

Do you use Test::Compile to check your code compiles correctly


Interesting, can you point me in how to use it?


hmm,  i didn't think that perl would be compiled? how does this work? /me is
interested as well
It's merely transparent for user, but there is an internal compilation 
phase before execution phase:

http://oreilly.com/catalog/pperl3/chapter/ch18.html#AUTOID-22329

Just try "perl -MAny::Module", it will break immediatly if the module 
doesn't compile correctly, even if you don't execute any code inside.

--
BOFH excuse #144:

Too few computrons available.


Re: [Mageia-dev] default $PATH after UsrMove

2012-11-12 Thread Colin Guthrie
'Twas brillig, and Luc Menut at 11/11/12 18:04 did gyre and gimble:
> I think that we should drop these dirs from our defaults $PATH, before a
> future mass rebuild.

Yeah seems sensible.

Col

-- 

Colin Guthrie
colin(at)mageia.org
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited http://www.tribalogic.net/
Open Source:
  Mageia Contributor http://www.mageia.org/
  PulseAudio Hacker http://www.pulseaudio.org/
  Trac Hacker http://trac.edgewall.org/


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Colin Guthrie
'Twas brillig, and Guillaume Rousse at 12/11/12 08:44 did gyre and gimble:
> Le 11/11/2012 19:13, r...@mageia.org a écrit :
>> +package LogViewer;
>> +
>> +use strict;
>> +use POSIX;
> never load the POSIX module this way. Either import it without importing
> any symbol in your namespace, either import just the symbols you need.
> See POSIX(3pm) for details.
> 
> [..]
>> +$::isWizard = 1;
> ugly syntax.
> 
> [..]
>> +#- check services
>> +my ) . Data::Dumper->Dump([ $service ], [qw(*services)]) . q(
>> +foreach (split(':', $options{SERVICES})) {
>> +next unless $services{$_};
>> +$r .= "Service $_ ($services{$_} is not running)\\n" unless -e
>> "/var/lock/subsys/$_";
>> +}
> You shouldn't leave debugging statement in your code. Moreover, I doubt
> this is syntactically correct. Do you use Test::Compile to check your
> code compiles correctly ?

Also /var/log/subsys is almost dead under systemd anyway. It's still
used by a few legacy initscripts that systemd calls, but I'm not sure
where this is going.

Also, if you are implementing a new log viewer module, is this going to
use the systemd journal rather than implementing it's own complex logic
for parsing compressed text files? I've not looked at the code, but it
would be *awesome* if this is what was happening :)

It's very easy to use the journal APIs (either by linking the library,
using python bindings - not sure if there are external perl bindings
yet, but I wouldn't be surprised if there are - or simply calling
journalctl on the command line and parsing the result. It offers an easy
to parse JSON format so even going via this route should be trivial if
official bindings do not exist). You can get all logs between certain
dates and for particular services with ease, without having to look in
the right file or grep for the right binary. i.e. all the building
blocks are in place for you :)

Col


-- 

Colin Guthrie
colin(at)mageia.org
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited http://www.tribalogic.net/
Open Source:
  Mageia Contributor http://www.mageia.org/
  PulseAudio Hacker http://www.pulseaudio.org/
  Trac Hacker http://trac.edgewall.org/


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread AL13N
Op maandag 12 november 2012 10:57:43 schreef Angelo Naselli:
> Il 12/11/2012 09:44, Guillaume Rousse ha scritto:
> > Do you use Test::Compile to check your code compiles correctly
> 
> Interesting, can you point me in how to use it?

hmm,  i didn't think that perl would be compiled? how does this work? /me is 
interested as well


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Steven Tucker

On 12/11/12 19:16, AL13N wrote:

Op maandag 12 november 2012 11:07:42 schreef Angelo Naselli:
[...]

Again I'm not a perl developer and neither i'm pretending to be,
what i'm trying to, is porting some old programs to new layout,
but if that hurts someone's feeling i can stop working on it
and do some in c++ for what i need.

[...]

Be advised that often people come accross as rude for certain people when it's
really just a different way of speaking in their primary language.

I doubt his feelings get hurt so easily though :-)

Also, look at it this way: if you really were bothering him, he wouldn't care
so much as to send detailed comments with where to find more info


This way of speaking (I find it is standard with technical people, not 
just a language issue) was a big shock to me the first time I offered up 
help, and seemed to be smashed back down.
I felt quite deflated and hurt, but it really is just getting to the 
point, and in most cases not meant as an insult but rather helping out 
by pointing out how things could be better. I learned to appreciate the 
fact that someone has bothered to analyse my code, regardless of their 
opinion of it.


At the end of the day, if you don't agree, you just don't change your 
code :-)


But just for the record ...

Angelo, you are doing a great job. I really appreciate your help (both 
hands on and advise) and I am sure most people here at least respect 
your effort, regardless of whether they agree with the project, or like 
the code itself. You did not initially want to use libyui and you did 
not want to use Perl. You are now doing both in the hope of achieving 
the greater goal.


Thank you!

Now get back and fix your rubbish code !! lol :-P

Cheers
Tuxta



Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Il 12/11/2012 09:44, Guillaume Rousse ha scritto:
> Do you use Test::Compile to check your code compiles correctly
Interesting, can you point me in how to use it?

Thanks
Angelo
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlCgyBcACgkQqEs9DA4DquDTAQCguRf0pLX7hVyMVmCk4ojrIpiR
LyIAnAp/bvAGZ4nllt/CnDAIfGulPNEy
=B9ED
-END PGP SIGNATURE-


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread AL13N
Op maandag 12 november 2012 11:07:42 schreef Angelo Naselli:
[...]
> Again I'm not a perl developer and neither i'm pretending to be,
> what i'm trying to, is porting some old programs to new layout,
> but if that hurts someone's feeling i can stop working on it
> and do some in c++ for what i need.
[...]

Be advised that often people come accross as rude for certain people when it's 
really just a different way of speaking in their primary language.

I doubt his feelings get hurt so easily though :-)

Also, look at it this way: if you really were bothering him, he wouldn't care 
so much as to send detailed comments with where to find more info


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Il 12/11/2012 10:06, Guillaume Rousse ha scritto:
> I forgot this one...
> 
> Le 11/11/2012 19:13, r...@mageia.org a écrit :
>> +## Append a custom string to log view adding date +sub logText
>> { +my ($st) = @_; +my $string = chomp_(`LC_ALL=C date
>> '+%Y %b %d %T'`) . " " . $st . "\n"; +# `` return non utf8
>> and concat of non utf8 & utf8 is non utf8: +
>> c::set_tagged_utf8($string); + +# log given text +
>> $logView->appendLines($string); +}
> It is especially ugly to call external date binary, whereas perl
> has everything needed as builtin functions for this. See "time
> related functions" in perlfunc(1), and POSIX::strftime in
> POSIX(3pm) if you need a simple formation function.
Ok so you're offering yourself to rewrite all without leaving
any old code in new implementation :D

Anyway I understand what you mean, and i agree i will fix it asap.

Angelo
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlCgy5sACgkQqEs9DA4DquDZ6ACeKl3WUjETSj6tFCXhTC9xox7W
BiQAn1sDo7GNTIdl7IgQu3q3UFrd/M3Y
=NCMa
-END PGP SIGNATURE-


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Angelo Naselli
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi Guillaume,
Il 12/11/2012 09:44, Guillaume Rousse ha scritto:
> Le 11/11/2012 19:13, r...@mageia.org a écrit :
>> +package LogViewer; + +use strict; +use POSIX;
> never load the POSIX module this way. Either import it without
> importing any symbol in your namespace, either import just the
> symbols you need. See POSIX(3pm) for details.

I thought it wasn't the right way, i just forgot to fix it.
This module is not completed yet, but feel free to work on it or
to any other module you want to port.

My basic idea was to show how an easy task like showing log could
work in qt, ncurses and gtk without changing the core code.

Again I'm not a perl developer and neither i'm pretending to be,
what i'm trying to, is porting some old programs to new layout,
but if that hurts someone's feeling i can stop working on it
and do some in c++ for what i need.

Funny from here:
> [..]
>> +$::isWizard = 1;
> ugly syntax.
> 
> [..]
>> +#- check services +my ) . Data::Dumper->Dump([ $service ],
>> [qw(*services)]) . q( +foreach (split(':', $options{SERVICES}))
>> { +next unless $services{$_}; +$r .= "Service $_
>> ($services{$_} is not running)\\n" unless -e 
>> "/var/lock/subsys/$_"; +}
> You shouldn't leave debugging statement in your code. Moreover, I
> doubt this is syntactically correct. Do you use Test::Compile to
> check your code compiles correctly ?
> 
the above code is all from old logdrake, all commented and not
runnable, because i haven't imported yet :p

Cheers,
Angelo
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlCgymsACgkQqEs9DA4DquDSBACggCiaEKouQ1s2wAFJHFmt1WBF
1joAoKFatBVmuK3ERVolx6B15XclNUtl
=qlDj
-END PGP SIGNATURE-


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Guillaume Rousse

I forgot this one...

Le 11/11/2012 19:13, r...@mageia.org a écrit :

+## Append a custom string to log view adding date
+sub logText {
+my ($st) = @_;
+my $string = chomp_(`LC_ALL=C date '+%Y %b %d %T'`) . " " . $st . "\n";
+# `` return non utf8 and concat of non utf8 & utf8 is non utf8:
+c::set_tagged_utf8($string);
+
+# log given text
+$logView->appendLines($string);
+}
It is especially ugly to call external date binary, whereas perl has 
everything needed as builtin functions for this. See "time related 
functions" in perlfunc(1), and POSIX::strftime in POSIX(3pm) if you need 
a simple formation function.

--
BOFH excuse #66:

bit bucket overflow


Re: [Mageia-dev] [changelog] [RPM] cauldron core/release gstreamer0.10-ffmpeg-0.10.13-5.mga3

2012-11-12 Thread Christiaan Welvaart

On Sun, 11 Nov 2012, Götz Waschk wrote:


On Sat, Nov 10, 2012 at 6:38 PM, Christiaan Welvaart
 wrote:

AFAIK it needs to be in the 'tainted' repository (MPEG4 AVC aka h.264
decoder).

the same code is in the main ffmpeg package as well, isn't it? So
either it has to be disabled in non-tainted build or moved to tainted
completely.


Oops, I guess you're right. I simply expected it to be disabled in the 
'core' ffmpeg build because of all the noise about this decoder wrt open 
source browsers (chromium has a bundled ffmpeg but the h.264 decoder is 
only enabled in 'chrome' branded builds).



Christiaan


Re: [Mageia-dev] [soft-commits] [6435] Added a first log viewer module.

2012-11-12 Thread Guillaume Rousse

Le 11/11/2012 19:13, r...@mageia.org a écrit :

+package LogViewer;
+
+use strict;
+use POSIX;
never load the POSIX module this way. Either import it without importing 
any symbol in your namespace, either import just the symbols you need. 
See POSIX(3pm) for details.


[..]

+$::isWizard = 1;

ugly syntax.

[..]

+#- check services
+my ) . Data::Dumper->Dump([ $service ], [qw(*services)]) . q(
+foreach (split(':', $options{SERVICES})) {
+next unless $services{$_};
+$r .= "Service $_ ($services{$_} is not running)\\n" unless -e 
"/var/lock/subsys/$_";
+}
You shouldn't leave debugging statement in your code. Moreover, I doubt 
this is syntactically correct. Do you use Test::Compile to check your 
code compiles correctly ?


--
BOFH excuse #208:

Your mail is being routed through Germany ... and they're censoring us.