Re: Elisa is in kdereview

2017-06-25 Thread Albert Astals Cid
El dimecres, 21 de juny de 2017, a les 23:15:58 CEST, Matthieu Gallien va 
escriure:
> Hello,
> 
> On mercredi 21 juin 2017 23:01:23 CEST Albert Astals Cid wrote:
> > El divendres, 16 de juny de 2017, a les 22:44:03 CEST, Matthieu Gallien va
> > 
> > escriure:
> > > Hello,
> > > 
> > > Elisa is now in kdereview and aiming for extragear/multimedia.
> > > 
> > > It is a music player from a design from Andrew Lake.
> > > It is using Qt multimedia for playback and a few KDE frameworks. Its UI
> > > is
> > > using Qml.
> > > 
> > > Its aim is to be simple to use and hopefully in the future powerfull
> > > when
> > > needed.
> > > 
> > > It should be usable without needing online services but will use them in
> > > the future.
> > > 
> > > A few integration bits are missing with respect to Baloo before I can do
> > > a
> > > release. Currently music can only be read if in its database that can be
> > > filled by Baloo or a custom file indexer if Baloo is not there.
> > 
> > That's kind of a weird design decision, basically i started elisa, it
> > didn't see any of my music, i didn't find a way to add it, so i removed
> > elisa.
> It is not a design decision but the current state. I want to also support
> the "let's play this file now" use case. I just had not yet enough time to
> do it.
> 
> I am also planning to add a "busy" indicator to show that Elisa is currently
> getting tracks from Baloo or default music directory (as per
> QStandardPaths). If no music is found, I also want to add a passive
> notification offering the possibility to configure the path to use to
> search music. I even have a task in phabricator for that.
> 
> > That'd be my workflow as a regular user.
> 
> I am already convinced that first impression is important. At the same time,
> I did request to move to extragear to get covered by CI and to get feedback
> from kde-core-devel.
> 
> > Cheers,
> > 
> >   Albert
> 
> [...]
> 
> By the way, did you build Elisa with Baloo ?

Yes

Cheers,
  Albert

> 
> Best regards




Re: kdereview - qtcurve

2017-06-25 Thread Albert Astals Cid
El dijous, 22 de juny de 2017, a les 16:07:01 CEST, David Edmundson va 
escriure:
> On Mon, Jun 19, 2017 at 11:42 PM, Albert Astals Cid  wrote:
> > El divendres, 16 de juny de 2017, a les 11:07:19 CEST, Yichao Yu va
> > 
> > escriure:
> > > QtCurve is now in kdereview with the intention of being in
> > > extragear/base
> > 
> > By default it assumes i want a Qt4 build and then fails because i don't
> > have
> > the necessary packages. Maybe it could be a bit gentler and just give me a
> > nice warning?
> 
> Possibly fixed. You'll need to wipe your CMakeCache though.

Not fixed for me https://paste.kde.org/pzgvpred2

Maybe the detection of KDE4 also needs to be optional and not REQUIRED?

Cheers,
  Albert

> 
> David




Re: Zanshin is in kdereview

2017-06-25 Thread Kevin Ottens
Hello,

On Thursday, 8 June 2017 09:36:51 CEST Kevin Ottens wrote:
> OK, this time it's the right one. :-)
> 
> Zanshin is now in kdereview and aiming for extragear/pim. Please review
> away!
> 
> Thanks in advance.

So I pushed just now all the patches mentioned in this thread. They should 
address all the feedback I got so far *except* the license "issue" (means it's 
effectively GPLv2) since I didn't get a reply from all the necessary people 
yet to allow relicensing. I don't think I can do much more about that one 
right now.

Any issue still preventing the move in extragear/pim?

Regards.
-- 
Kévin Ottens, http://ervin.ipsquad.net

KDAB - proud supporter of KDE, http://www.kdab.com



signature.asc
Description: This is a digitally signed message part.


Re: kdereview - qtcurve

2017-06-25 Thread Albert Astals Cid
El divendres, 16 de juny de 2017, a les 11:07:19 CEST, Yichao Yu va escriure:
> QtCurve is now in kdereview with the intention of being in extragear/base

For the record, I also asked Yichao to have a look at a comment one of our 
translators did in the i18n mailing list (via a CC'ing), haven't heard back 
about it yet, so i'm mentioning it here too.

Cheers,
  Albert