All of lore.kernel.org
 help / color / mirror / Atom feed
From: Till Kamppeter <till.kamppeter@gmail.com>
To: "Hal V. Engel" <hvengel@gmail.com>
Cc: Printing-architecture@lists.linux-foundation.org,
	Lars Uebernickel <larsuebernickel@gmx.de>
Subject: Re: [Printing-architecture] Qt only CPD.
Date: Sat, 26 Feb 2011 22:49:54 +0100	[thread overview]
Message-ID: <4D697582.7080005@gmail.com> (raw)
In-Reply-To: <201102261340.55287.hvengel@gmail.com>

On 02/26/2011 10:40 PM, Hal V. Engel wrote:
> Thanks that helped. I think I commited my current changes. At least the
> messages I got looked like it worked. But it does not apear to show up
> in the web interface to the repository. But perhaps that needs some time
> to sync up.

Up to now nothing arrived. Did you use "bzr push ..." or the Patch Queue 
Manager (PQM)? Note that the requests send to the patch queue manager 
are not applied immediately, but by a cron job. Perhaps you only got 
confirmation of the submission and not of the actual application of your 
changes.

> Please feel free to give the dialog a test run if it
> actually made it into the repository. The dialog looks and behaves just
> like the original version for the most part. Things related to this that
> still need to be done:
>

I will try the dialog a soon as I find it on the BZR.

>
> 1. I did not do much with the cmake setup as I am far from an expert on
> how cmake does things and I wanted to get the rest of this checked in.
> So this still needs to be modified to remove kde references.
>
>
> 2. I had to add some icon files that where apparently included in KDE.
> An application resource needs to be created for these and the code
> modified to use the resource instead of the files.
>
>
> 3. The directory should probably be renamed from kde4-dialog to qt4-dialog.

Can you please do these corrections to complete the change to Qt-only? I 
do not know, but is cmake KDE-specific or can it also be used for a 
non-KDE application like the pure Qt CPD?

    Till

      reply	other threads:[~2011-02-26 21:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-26  4:00 [Printing-architecture] Qt only CPD Hal V. Engel
2011-02-26 13:02 ` Till Kamppeter
2011-02-26 16:16   ` Hal V. Engel
2011-02-26 18:48     ` Till Kamppeter
2011-02-26 21:40       ` Hal V. Engel
2011-02-26 21:49         ` Till Kamppeter [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4D697582.7080005@gmail.com \
    --to=till.kamppeter@gmail.com \
    --cc=Printing-architecture@lists.linux-foundation.org \
    --cc=hvengel@gmail.com \
    --cc=larsuebernickel@gmx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.