Yes, backwards compatibility in fact was explicitely meant to be broken.
Agreed on lxqt / lxqt.
J. Leclanche
On Wed, Aug 7, 2013 at 6:03 AM, [email protected] <[email protected]
> wrote:
> 2013/8/7 Kuzma Shapran <[email protected]>
>
>> My opinion:
>> 1. lxqt
>> 2. lxqt
>> 3. I think we discussed that we do not provide backward compatibility.
>> 4. It's really good to use C++ namespace rather than C-style prefix, so:
>> lxqt::
>>
>
> I agree: lxqt, lxqt and lxqt:: . And we do not need to be backward
> compatible.
>
> br. Chr.
>
>
>> 5. I just checked and really found 1 file (razorshortcutbutton_p.h) in
>> librazorqt - it's not used and it had to be removed with other old files
>> when the current global shortcuts were implemented. so ignore it.
>> "razor-global-key-shortcut-selector" library contains a widget to select
>> a shortcut - it's used in config dialogs of apps and plugins which support
>> global shortcuts, it's also used in global shortcut config app:
>> "razorqt-globalkeyshortcuts/config".
>> "razor-global-key-shortcuts-client" - is a library which is actually
>> connects to global shortcut daemon. it's used in apps and plugins which
>> support global shortcuts, but not used in the config app.
>> They potentially could be merged, but "razor-global-key-shortcuts-client"
>> does not requires QtGui/QtWidgets, while
>> "razor-global-key-shortcut-selector" is purely a UI widget, so in short
>> words - it's better to keep them separately, I think.
>>
>> Cheers,
>> Kuzma
>>
>> P.S. I had a problem with naming this global shortcut system - I don't
>> really like this huge "razor-global-key-shortcuts-...". Any naming
>> suggestions are welcome!
>>
>>
>> On 7 August 2013 14:21, PCMan <[email protected]> wrote:
>>
>>> On Tue, Aug 6, 2013 at 11:30 PM, Александр Соколов <[email protected]>
>>> wrote:
>>> > I changed libqtxdg now it provide cmake find file. So cmake for the
>>> > application what use qtxdg should be like
>>> >
>>> > find_package(QTXDG REQUIRED)
>>> > include(${QTXDG_USE_FILE})
>>> >
>>> > worked example
>>> >
>>> https://github.com/Razor-qt/razor-qt/blob/separate-build/razorqt-confupdate/CMakeLists.txt
>>> > You can copy razorqt-confupdate into separate dir and build it.
>>> >
>>> >
>>> > 2013/8/6 PCMan <[email protected]>
>>> >>
>>> >> I just made liblxqt (formerly librazorqt) compile independently.
>>> >> The fix is a little bit ugly (copy and paste from the toplevel
>>> >> CMakeLists.txt).
>>> >> Anyway, it works.
>>> >> I haven't change its binary name, so it's still librazorqt now.
>>> >> Later we can rename it, and start fixing other components depending
>>> on it.
>>> >> I'll start with some less important stuff, such as razor-about and
>>> >> razor-policykit.
>>> >> I won't touch the panel, which is by far the most complicated thing,
>>> >> at the moment.
>>> >>
>>> > I have such patches for librazor, let's I change it.
>>> >
>>> > --
>>> > Best regards,
>>> > Alexander.
>>>
>>> Cool! I just tested libqtxdg and it compiles well.
>>> There are some issues I'd like to ask for opinions.
>>>
>>> 1. component naming:
>>> Should we use "lxqt-" prefix proposed earlier or it's better to use
>>> "lxdeqt-"? It's a little weird sometimes that the binary name is
>>> different from project name.
>>>
>>> 2. dir naming:
>>> Should the config and data dirs be named lxqt, or lxdeqt, or lxde-qt?
>>>
>>> 3. API naming:
>>> Should we retain Razor* names for all of the APIs for backward
>>> compatibility, or we need to rename them to LxQt* and do typedef for
>>> Razor ones for backward compatibility?
>>>
>>> 4. C++ Namespace:
>>> Should we prefix every class name with "Razor" (or "LxQt", "LxdeQt"),
>>> or we should use C++ namespace? (for ex: Razor::Panel or
>>> LxdeQt::Panel). We used C++ namespace in lxde-qt, but if you prefer
>>> the current way, I'm OK with adding prefix to every class name and
>>> ignore C++ namespace, too.
>>>
>>> 5. There seems to be different parts of global shortcut stuff put
>>> separately in three places. librazorqt, and the other 2 libs in
>>> razor-qt/libraries. What's the relationship among them? Can we
>>> simplify this and put them in one single component?
>>>
>>> Comments are wanted so we can continue.
>>> Thanks a lot!
>>>
>>>
>>> ------------------------------------------------------------------------------
>>> Get 100% visibility into Java/.NET code with AppDynamics Lite!
>>> It's a free troubleshooting tool designed for production.
>>> Get down to code-level detail for bottlenecks, with <2% overhead.
>>> Download for free and get started troubleshooting in minutes.
>>>
>>> http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
>>> _______________________________________________
>>> Lxde-list mailing list
>>> [email protected]
>>> https://lists.sourceforge.net/lists/listinfo/lxde-list
>>>
>>
>> --
>> --
>> You received this message because you are subscribed to the Google
>> Groups "Razor-qt" group.
>> For more options, visit this group at
>> http://groups.google.com/group/razor-qt?hl=en
>>
>> ---
>> You received this message because you are subscribed to the Google Groups
>> "Razor-qt" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to [email protected].
>> For more options, visit https://groups.google.com/groups/opt_out.
>>
>>
>>
>
> --
> --
> You received this message because you are subscribed to the Google
> Groups "Razor-qt" group.
> For more options, visit this group at
> http://groups.google.com/group/razor-qt?hl=en
>
> ---
> You received this message because you are subscribed to the Google Groups
> "Razor-qt" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> For more options, visit https://groups.google.com/groups/opt_out.
>
>
>
------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead.
Download for free and get started troubleshooting in minutes.
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
Lxde-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lxde-list
