0.10.0-beta1 release tarball...

classic Classic list List threaded Threaded
27 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Marcel Wiesweg

> Aha. This is not tarball (sorry) but svn version.

Even better

>
> Looks very good.
>
> Some problems (in random order):
>
> 1. In image plugins there are no target image previews, only originals.
> 2. Panels are too wide.

I can confirm, not acceptable minimum sidebar width here too.
There are widgets in there which have a too wide minimum size. I suspect
Marble as both left and right sidebar is affected, but we need to
investigate.

> 3. How to perform Fuzzy Search? When I open tab Album are clears and
>    don't know how to add image as a pattern.

Hehe. You can drag and drop it the whole way ;-) I guess we should add a
context menu option.

> 4. When creating new Album there is still Collections input field.
> 5. When moving album by drag'n'drop looks like moved folder overwrites
>    folder where it was moved on. Eg. when moving A to B, only A stays
>    visible while it should create B->A hierarchy. On disk is everything
>    OK and after restart hierarchy looks good.
> 6. Similar problem when deleting album, tree isn't refreshed

I need to look if all signals are properly connected

> 7. Cannot select images with mouse (drawing rectangle)

There are issues with QRubberband. Both Gilles (I think) and I are a bit
unnerved about this. QRubberband works probably well with the new
QScrollAreas, but needs some work in our Q3ScrollView environments.
Note the Left/Top to Bottom/Right works, even if the rubber band is shown only
the first time. Same in IE.

> 8. In advanced search "Remove group" doesn't look clickable.

I have made it italic and underlined now.

>
> Thats for now.

Thanks for your valuable feedback!

>
> Overall very good impression. Visually new features (like Advanced
> Search) are looking better than old, being designed from scratch for
> Oxygen not only ported.
>
> m.
>
> _______________________________________________
> Digikam-devel mailing list
> [hidden email]
> https://mail.kde.org/mailman/listinfo/digikam-devel


_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball... - undefined reference to `pthread_create'

Mark Purcell
In reply to this post by Mark Purcell
On Sat, 5 Jul 2008, Mark Purcell wrote:
> > digikam-0.10.0~beta1/libs/greycstoration/greycstoration.h:218:
> > undefined reference to `pthread_create'

> it doesn't appear to be linking -lpth in :-(

I have this issue with both the beta1 tarball and building direct from
svn. :-(

> -lkdecore -lkio -lkhtml -lkdeui -lkparts -lkio -lkdeui -lQtGui -lQtSvg
> -lkdecore -lsolid -lQtCore -lQtNetwork -lQtDBus -lQtXml -lkutils
> -lkde3support -lQt3Support -ljpeg -ltiff -lpng -lz -llcms -ljasper -ljpeg
> -ltiff -lpng -lz -llcms -ljasper -lkdcraw -lkexiv2 -lkipi -lmarblewidget
> -lkdeui -lsolid -lkparts -lkdecore -lkdeui -lkparts
[...]
> undefined reference to `pthread_create'

I am able to build if I manually add a "-lpthread" to the end of the line in:
build/digikam/digikam/CMakeFiles/digikam.dir/link.txt

> Am I missing some CMakeLists.txt foo?

I don't know my way around CMake, but from the above it looks like the
CMakeLists.txt needs to search and include libpthread.

Mark

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

signature.asc (204 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Bugzilla from mikmach@wp.pl
In reply to this post by Marcel Wiesweg
Dnia Sunday 06 of July 2008, Marcel Wiesweg napisaƂ:
> > 3. How to perform Fuzzy Search? When I open tab Album are clears and
> >    don't know how to add image as a pattern.
>
> Hehe. You can drag and drop it the whole way ;-) I guess we should add a
> context menu option.

Not sure about context menu option. It is already crowded and I could
find better candidates to add new item there. It is in main menus but
IMO in bad place - in Image section. All other searches are in Tools.

> > 7. Cannot select images with mouse (drawing rectangle)
>
> There are issues with QRubberband. Both Gilles (I think) and I are a bit
> unnerved about this. QRubberband works probably well with the new
> QScrollAreas, but needs some work in our Q3ScrollView environments.
> Note the Left/Top to Bottom/Right works, even if the rubber band is
> shown only the first time. Same in IE.

Yes, Left/Top -> Bottom/Right works.

> > Thats for now.
>
> Thanks for your valuable feedback!

Few more glitches:

1. In context menu icons for elements are ugly imposed over left side
   decoration
2. Button Search in toolbar should switch to appropriate panel or
   should be removed - there is easy access to it. Ar maybe make it for
   Advanced search?
3. No feedback (at least I didn't notice) when fuzzy search finds
   nothing.

m.


_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Gilles Caulier-4
In reply to this post by Gilles Caulier-4
Hi al,

I have updated the 0.10.0-beta1 tarball with all changes done during
this week end.

http://digikam3rdparty.free.fr/0.10.x-releases/digikam-0.10.0-beta1.tar.bz2

Please give me a feedback if all compile fine on your computers before
to make the official release.

Thanks in advance

Gilles Caulier

Note : there is also an online page with translations stats:

http://digikam3rdparty.free.fr/0.10.x-releases/digikam-l10n-0.10.0-beta1.html
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Mark Purcell
On Mon, 7 Jul 2008, Gilles Caulier wrote:
> Please give me a feedback if all compile fine on your computers before
> to make the official release.

Gilles,

This version still fails to link using -lpthread for me on Debian which
results in:
        undefined reference to `pthread_create'.

As I mentioned yesterday I can workaround by manually patching links.txt,
during build.

Before building I have also found out how to force this inclusion in
TARGET_LINK_LIBRARIES in digikam/CMakeLists.txt:

--- digikam/CMakeLists.txt.orig 2008-07-07 22:21:31.000000000 +1000
+++ digikam/CMakeLists.txt      2008-07-07 21:57:05.000000000 +1000
@@ -117,6 +117,7 @@
                       ${KDCRAW_LIBRARIES}
                       ${KEXIV2_LIBRARIES}
                       ${KIPI_LIBRARIES}
+                      ${CMAKE_THREAD_LIBS_INIT}
               )

 IF(MARBLEWIDGET_FOUND)

I suspect that this should automatically be included, and that is why others
are having success.  But for the Debian builds it appears that -lpthread
isn't included and the binary doesn't link.
${CMAKE_THREAD_LIBS_INIT} evaluates to "-lpthread".

I am unsure if this is going to effect anything else though as I don't grok
CMake.

Mark

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

signature.asc (204 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Gilles Caulier-4
2008/7/7 Mark Purcell <[hidden email]>:

> On Mon, 7 Jul 2008, Gilles Caulier wrote:
>> Please give me a feedback if all compile fine on your computers before
>> to make the official release.
>
> Gilles,
>
> This version still fails to link using -lpthread for me on Debian which
> results in:
>        undefined reference to `pthread_create'.
>
> As I mentioned yesterday I can workaround by manually patching links.txt,
> during build.
>
> Before building I have also found out how to force this inclusion in
> TARGET_LINK_LIBRARIES in digikam/CMakeLists.txt:
>
> --- digikam/CMakeLists.txt.orig 2008-07-07 22:21:31.000000000 +1000
> +++ digikam/CMakeLists.txt      2008-07-07 21:57:05.000000000 +1000
> @@ -117,6 +117,7 @@
>                       ${KDCRAW_LIBRARIES}
>                       ${KEXIV2_LIBRARIES}
>                       ${KIPI_LIBRARIES}
> +                      ${CMAKE_THREAD_LIBS_INIT}
>               )
>
>  IF(MARBLEWIDGET_FOUND)
>
> I suspect that this should automatically be included, and that is why others
> are having success.  But for the Debian builds it appears that -lpthread
> isn't included and the binary doesn't link.
> ${CMAKE_THREAD_LIBS_INIT} evaluates to "-lpthread".
>
> I am unsure if this is going to effect anything else though as I don't grok
> CMake.
>
> Mark

Perhaps a Debian expert can confirm a bug in CMake included in current
version ? Achim ?

Gilles Caulier
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: 0.10.0-beta1 release tarball...

Arnd Baecker


On Mon, 7 Jul 2008, Gilles Caulier wrote:

> 2008/7/7 Mark Purcell <[hidden email]>:
> > On Mon, 7 Jul 2008, Gilles Caulier wrote:
> >> Please give me a feedback if all compile fine on your computers before
> >> to make the official release.
> >
> > Gilles,
> >
> > This version still fails to link using -lpthread for me on Debian which
> > results in:
> >        undefined reference to `pthread_create'.
> >
> > As I mentioned yesterday I can workaround by manually patching links.txt,
> > during build.
> >
> > Before building I have also found out how to force this inclusion in
> > TARGET_LINK_LIBRARIES in digikam/CMakeLists.txt:
> >
> > --- digikam/CMakeLists.txt.orig 2008-07-07 22:21:31.000000000 +1000
> > +++ digikam/CMakeLists.txt      2008-07-07 21:57:05.000000000 +1000
> > @@ -117,6 +117,7 @@
> >                       ${KDCRAW_LIBRARIES}
> >                       ${KEXIV2_LIBRARIES}
> >                       ${KIPI_LIBRARIES}
> > +                      ${CMAKE_THREAD_LIBS_INIT}
> >               )
> >
> >  IF(MARBLEWIDGET_FOUND)
> >
> > I suspect that this should automatically be included, and that is why others
> > are having success.  But for the Debian builds it appears that -lpthread
> > isn't included and the binary doesn't link.
> > ${CMAKE_THREAD_LIBS_INIT} evaluates to "-lpthread".
> >
> > I am unsure if this is going to effect anything else though as I don't grok
> > CMake.
> >
> > Mark
>
> Perhaps a Debian expert can confirm a bug in CMake included in current
> version ? Achim ?

You could also just fetch the binary cmake 2.6 from
http://www.cmake.org and try that ...

Best, Arnd
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
12