[PACKAGED] qt-update-notifer - Is it OK?

[sling-shot]

qt-update-notifier
---------------------------
This project seeks to provide a kde technology based pclinuxos update notifier tray application.
https://github.com/mhogomchungu/qt-update-notifier#readme
License GPLv2 or higher

This is essentially a replacement for our existing GTK based update notifier written in Qt by forum member muungwana

I have done a preliminary package on my own and it seems to install and run OK on my test system. (I have not been able to test if it informs of updates properly due to lack of time)

Below are the links and I request feedback.

NOTE : This is a 32 bit only build.

SPEC - http://dl.dropbox.com/u/37931428/qt-update-notifier/pclos-qt-update-notifier.spec
LOG - http://dl.dropbox.com/u/37931428/qt-update-notifier/pclos-qt-update-notifier.spec.log
RPM - http://dl.dropbox.com/u/37931428/qt-update-notifier/qt-update-notifier-1.2.0-1slingshot2013.i586.rpm
SRPM - http://dl.dropbox.com/u/37931428/qt-update-notifier/qt-update-notifier-1.2.0-1slingshot2013.src.rpm

Thank you very much muungwana

[ghostbunny]

Hi sling-shot,

one little thing in the spec please replace libqt4-devel by %{_lib}qt4-devel so it should be buildable from this spec on 64bit also.

i didnt test the program itself i don't want to mess up my lxde

but the build log looks good for me.

regards
ghostbunny

[TerryN]

Although it doesn't make much difference for this (small) application you should really use:

Code: [Select]

%cmake
rather than:

Code: [Select]

cmake -DCMAKE_INSTALL_PREFIX=`kde4-config --prefix`  -DCMAKE_BUILD_TYPE=release
This ensures that all the compiler and linker flags are set correctly as well as prefixes for both 32bit and 64bit and other standard paths.

Terry.

[sling-shot]

one little thing in the spec please replace libqt4-devel by %{_lib}qt4-devel so it should be buildable from this spec on 64bit also.

Can this be followed as a rule of thumb in general?

[sling-shot]

Although it doesn't make much difference for this (small) application you should really use:

Code: [Select]

%cmake
rather than:

Code: [Select]

cmake -DCMAKE_INSTALL_PREFIX=`kde4-config --prefix`  -DCMAKE_BUILD_TYPE=release
This ensures that all the compiler and linker flags are set correctly as well as prefixes for both 32bit and 64bit and other standard paths.

Terry.

I copied that line from the build instructions given in the source. Thanks for correcting me.

--------------

Also, the instruction in the source was to use "make install" but using that produced error in not being able to copy the .png images from build to buildroot.
I tried the alternative %makeinstall_std which I had seen being used elsewhere and it worked.
(Do not know the difference)

[ghostbunny]

one little thing in the spec please replace libqt4-devel by %{_lib}qt4-devel so it should be buildable from this spec on 64bit also.

Can this be followed as a rule of thumb in general?

no because there are still some lib/lib devel packages in 64bit repo which do not have the 64 tag but in the case of libqt4-devel it is %{_lib}, always. which means there exists a lib64qt4-devel

[ghostbunny]

Also, the instruction in the source was to use "make install" but using that produced error in not being able to copy the .png images from build to buildroot.
I tried the alternative %makeinstall_std which I had seen being used elsewhere and it worked.
(Do not know the difference)

definition of %makeinstall_std

Code: [Select]

%makeinstall_std make DESTDIR=%{?buildroot:%{buildroot}} install

[TerryN]

Can this be followed as a rule of thumb in general?

no because there are still some lib/lib devel packages in 64bit repo which do not have the 64 tag but in the case of libqt4-devel it is %{_lib}, always. which means there exists a lib64qt4-devel

You should look at the "provides" of the -devel package to see if there is an architecture neutral virtual provide for the lib and use that where possible.
For example:

Code: [Select]

[terry@localhost ~]$ rpm -q --provides lib64qt4-devel
qt-devel = 4.8.4-2pclos2013
qt4-devel = 4.8.4-2pclos2013
libqt4-devel = 4.8.4-2pclos2013
...

So you could use qt-devel, qt4-devel or libqt4-devel.  If there is no architecture neutral VP then you will need to use the %{_lib}qt4-devel format.

Terry.

[muungwana]

thank you sling-shot for packaging this,very much appreciate it.

[gseaman]

I think that it should be in the 'Software Center' submenu. I found it in 'File Tools'. I also cannot make out what the icons are supposed to represent with a 28 pixel panel. Otherwise, everything seems to work as expected.

Galen

[sling-shot]

I think that it should be in the 'Software Center' submenu. I found it in 'File Tools'. I also cannot make out what the icons are supposed to represent with a 28 pixel panel. Otherwise, everything seems to work as expected.

Galen

muungwana is watching this thread. Let me wait for him to update this at source rather than me patching it.

I had just used the .desktop file that came with the source. While testing I did not go through menu but just used the search feature.

[muungwana]

I think that it should be in the 'Software Center' submenu. I found it in 'File Tools'. I also cannot make out what the icons are supposed to represent with a 28 pixel panel. Otherwise, everything seems to work as expected.

Galen

The icon reads "uN" for update notifier.
The important think about the icon is its color since thats the primary way it communicates with the user.

It works as following.

After it is started,it waits for 5 minutes before checking if its time to check for updates or not.
By defaul,it will try to check for updates only after atleast 24 hours have passed since the last check.
The default state where there are no updated has the icon in greenish color.

If there are no updates,the icon will be in "passive state" and hence an "auto" system tray option for it will have it hidden.

If if finds updates,the icon changes to reddish color and will be in "need attention state" and an "auto" system tray option will have the icon displayed ie,not hidden.

The icon changes its color to a whitish when its checking for updates

The tool tip will also tell you if there are no updates or if there are updates.

Right click on the icon and should see various options.You can check for logs to see what is happening.

[muungwana]

just looked and it has this in the .desktop file

Categories=Utility;Qt;X-MandrivaLinux-System-FileTools;

I based the .desktop file from zuluCrypt one and forgot to change that file.

What should the category line be to have the icon in a proper place?

[gseaman]

Maybe?

Categories=PackageManager;System;Application;X-MandrivaLinux-System-Configuration;X-MandrivaLinux-System-Configuration-Packaging;

Galen

[muungwana]

Maybe?

Categories=PackageManager;System;Application;X-MandrivaLinux-System-Configuration;X-MandrivaLinux-System-Configuration-Packaging;

Galen

Thanks,i made the change and make a new release.
The new sources can be found at: https://github.com/mhogomchungu/qt-update-notifier/tree/v1.1.1

You can get them the same way,just click the "zip" button on the page.