PCMan File Manager

Le 05/22/2012 11:30 PM, Andrej N. Gritsenko a écrit :
>     That doesn't include libfm and pcmanfm obviously
There are included :
https://launchpad.net/~lubuntu-dev/+archive/lubuntu-daily/+packages?field.name_filter=fm&field.status_filter=published&field.series_filter=precise

And I remove all the patches, on purpose, to be sure to have a pristine
build. We include some patches in Ubuntu packages that should be
includes in git directly, but PCMan didn't merge them yet.

Regards,
Julien Lavergne

Hello!

Julien Lavergne has written on Tuesday, 22 May, at 23:43:
>Le 05/22/2012 11:30 PM, Andrej N. Gritsenko a écrit :
>>     That doesn't include libfm and pcmanfm obviously
>There are included :
>https://launchpad.net/~lubuntu-dev/+archive/lubuntu-daily/+packages?field.name_filter=fm&field.status_filter=published&field.series_filter=precise

    I see, so you have different Git then. Something is wrong it seems
and needs to be changed.

>And I remove all the patches, on purpose, to be sure to have a pristine
>build. We include some patches in Ubuntu packages that should be
>includes in git directly, but PCMan didn't merge them yet.

    Add changes in your debian/rules file, please, or it wouldn't allow
usage of multicore CPU and always compiles in one thread. What I changed:

target %:
    +dh intead of dh.
new target override_dh_auto_build:
    $(MAKE)

    With best wishes.
    Andiy.

Hello!

Julien Lavergne has written on Tuesday, 22 May, at 23:43:
>Le 05/22/2012 11:30 PM, Andrej N. Gritsenko a écrit :
>>     That doesn't include libfm and pcmanfm obviously
>There are included :
>https://launchpad.net/~lubuntu-dev/+archive/lubuntu-daily/+packages?field.name_filter=fm&field.status_filter=published&field.series_filter=precise

    I've tried to build libfm from official Git sources with debian/
directory from mentioned PPA and got few pages of warnings. JFYI.

    Cheers!
    Andriy.

On Wed, May 23, 2012 at 5:43 AM, Julien Lavergne <gilir@...> wrote:

> Le 05/22/2012 11:30 PM, Andrej N. Gritsenko a écrit :
> >     That doesn't include libfm and pcmanfm obviously
> There are included :
>
> https://launchpad.net/~lubuntu-dev/+archive/lubuntu-daily/+packages?field.name_filter=fm&field.status_filter=published&field.series_filter=precise
>
> And I remove all the patches, on purpose, to be sure to have a pristine
> build. We include some patches in Ubuntu packages that should be
> includes in git directly, but PCMan didn't merge them yet.
>
> Would you please tell me which patches are not merged?
IIRC, I reviewed the patches you used last week and most of them are no
longer needed.
If I did not merge a patch, there should be a good reason or I just forgot
to do it.
Thanks

Regards,
> Julien Lavergne
>
>
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> Pcmanfm-develop mailing list
> Pcmanfm-develop@...
> https://lists.sourceforge.net/lists/listinfo/pcmanfm-develop
>

Hello!

PCMan has written on Wednesday, 23 May, at  7:43:
> Would you please tell me which patches are not merged?

>IIRC, I reviewed the patches you used last week and most of them are no
>longer needed.
>If I did not merge a patch, there should be a good reason or I just forgot
>to do it.

    Sorry for intrusion. What I've found in source for pcmanfm currently
in Ubuntu Precise (which aren't in Git on SF) are:

01-libfm-0.1.14-API-changes.patch -- it replaces obsolete calls
    fm_path_new() with fm_path_new_for_str().

91-integrate-lxshortcut.patch -- it adds one more item in menu when
    right-clicked on empty space in directory view: 'Shortcut', which
    calls lxshortcut to make a new .desktop file in the directory.
    That is very handy BTW.

Haven't explored which patches there is in that PPA.

    With best wishes.
    Andriy.

Le 05/23/2012 01:43 AM, PCMan a écrit :
> Would you please tell me which patches are not merged?
> IIRC, I reviewed the patches you used last week and most of them are
> no longer needed.
> If I did not merge a patch, there should be a good reason or I just
> forgot to do it.
> Thanks

This is the remaining patches :

libfm :
- 02-libfm-0.1.14-API-changes.patch : Re-enable API which is still used
in pcmanfm (could be dropped when pcmanfm doesn't use this API anymore,
see patch in pcmanfm)
- 05_set_icon_name_for_main_window.patch : Set icon to preferred
application windows (re-done to apply to the .glade file).

pcmanfm :
- 01-libfm-0.1.14-API-changes.patch : Finish migration to the new API
- 02-integrate-lxshortcut.patch : Integrate the ability to add a
shortcut after a right click on the desktop
- 03_avoid_free.patch : Fix a possible crash

All others patches have been dropped or merged.

Regards,
Julien Lavergne