Launchpad has imported 12 comments from the remote bug at
https://bugzilla.xfce.org/show_bug.cgi?id=9813.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2013-01-28T15:07:59+00:00 Christian Hesse wrote:

When printing with mousepad I get line numbers on the paper though this
option is disable in printing options.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/0

------------------------------------------------------------------------
On 2013-01-28T15:08:41+00:00 Christian Hesse wrote:

Btw, tested with latest git (0.3.0-12-g8d234ab).

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/1

------------------------------------------------------------------------
On 2013-05-11T10:49:58+00:00 b3nmore wrote:

My guess: gtk_source_print_compositor_set_print_line_numbers () is not
set to 0 if 'print line numbers' is unchecked. In both cases the value
of line_number_increment is used, which is always in [1,100].

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/2

------------------------------------------------------------------------
On 2013-06-05T11:42:34+00:00 b3nmore wrote:

Created attachment 5056
Should fix, that line numbers are always printed

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/3

------------------------------------------------------------------------
On 2013-06-06T15:43:56+00:00 b3nmore wrote:

Created attachment 5057
Fixes: line numbers are always printed

Additionally set print->line_number_increment = 0, since this is stored
and restored when printing the next time.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/4

------------------------------------------------------------------------
On 2013-07-24T16:51:17+00:00 b3nmore wrote:

Created attachment 5103
Should fix: line numbers are always printed

Second approach was too simple and left
gtk_source_print_compositor_set_print_line_numbers in some cases in
undefined/wrong state. The attached patch hopefully handles this more
robust.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/5

------------------------------------------------------------------------
On 2013-10-14T03:57:46+00:00 Andreldm1989-j wrote:

(In reply to b3nmore from comment #5)
> Created attachment 5103 [details]
> Should fix: line numbers are always printed
> 
> Second approach was too simple and left
> gtk_source_print_compositor_set_print_line_numbers in some cases in
> undefined/wrong state. The attached patch hopefully handles this more robust.

Sorry, but the patch doesn't solve the problem here.
Also notice that the patch tries to rename the filename and has to be applied 
on the mousepad/mousepad folder(where the source files are), I think it should 
be applied on mousepad(where the configs files are).
I'll try to investigate this problem soon.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/6

------------------------------------------------------------------------
On 2013-10-14T15:09:06+00:00 b3nmore wrote:

(In reply to André Miranda from comment #6)
> Sorry, but the patch doesn't solve the problem here.
> Also notice that the patch tries to rename the filename ...
Hm, this is strange. How did you apply the patch exactly?
> ... and has to be
> applied on the mousepad/mousepad folder(where the source files are), ...
Yes, this how patches usually are used. You change (in this case) 
mousepad-print.c by applying the patch to it. Then you compile mousepad and the 
resulting binary shouldn't show the corresponding issue anymore.
> ... I think
> it should be applied on mousepad(where the configs files are).
No idea, what you mean by this.
> I'll try to investigate this problem soon.
Good luck.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/7

------------------------------------------------------------------------
On 2013-10-14T21:49:34+00:00 Andreldm1989-j wrote:

(In reply to b3nmore from comment #7)
> (In reply to André Miranda from comment #6)
> > Sorry, but the patch doesn't solve the problem here.
> > Also notice that the patch tries to rename the filename ...
> Hm, this is strange. How did you apply the patch exactly?
git apply ~/mousepad-print.patch
But I must have done something wrong, now the fix seems to be working just 
nicely :)

> > ... and has to be
> > applied on the mousepad/mousepad folder(where the source files are), ...
> Yes, this how patches usually are used. You change (in this case)
> mousepad-print.c by applying the patch to it. Then you compile mousepad and
> the resulting binary shouldn't show the corresponding issue anymore.
> > ... I think
> > it should be applied on mousepad(where the configs files are).
> No idea, what you mean by this.
I meant the following:
git clone <url> mousepad
cd mousepad (local repo root)
cd mousepad (this folder is known in other projects as 'src')

See? there are 2 mousepad folders, in order to your patch be applied I must do 
the git apply on the second folder.
Also notice the first two lines on the patch, git cannot find 
"print_fix_num_printing.c". I hope now you understand what I mean :)

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/8

------------------------------------------------------------------------
On 2013-10-15T15:55:53+00:00 b3nmore wrote:

Alright, I used good ol' diff to create the patch. So it was meant to be
applied with patch. I should use git format-patch in the future ...

Anyway, glad to hear that it works for you now.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/9

------------------------------------------------------------------------
On 2014-07-12T03:21:42+00:00 G-matt-8 wrote:

Patch applied in
http://git.xfce.org/apps/mousepad/commit/?id=8525dbdaf58cce86f235ec92923e1c48f493c6d9.
Thanks!

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/13

------------------------------------------------------------------------
On 2014-07-12T03:23:47+00:00 G-matt-8 wrote:

Ah crap. I credited Christian Hesse in the commit message instead of
b3nmore! Sorry about that b3nmore, unfortunately I can't change it I
don't think.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/mousepad/+bug/1267958/comments/14


** Changed in: mousepad
       Status: Unknown => Fix Released

** Changed in: mousepad
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1267958

Title:
  mousepad always prints line numbers

To manage notifications about this bug go to:
https://bugs.launchpad.net/mousepad/+bug/1267958/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to