Merge lp:~paulbrianstewart/ubuntu/oneiric/visualvm/813165-formattingTypo into lp:ubuntu/oneiric/visualvm

Proposed by Paul Stewart
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~paulbrianstewart/ubuntu/oneiric/visualvm/813165-formattingTypo
Merge into: lp:ubuntu/oneiric/visualvm
Diff against target: 19 lines (+4/-4)
1 file modified
debian/control (+4/-4)
To merge this branch: bzr merge lp:~paulbrianstewart/ubuntu/oneiric/visualvm/813165-formattingTypo
Reviewer Review Type Date Requested Status
Paul Stewart (community) Needs Resubmitting
Luke Yelavich (community) Approve
Matthew Paul Thomas (community) design Approve
Ubuntu branches Pending
Review via email: mp+68469@code.launchpad.net

Commit message

  Changed the bullet point formatting. This was formatted incorrectly and causing the first line to delete the first 2 characters of the word 'VisualVM....'
  I added 's' to the end of the words 'Application Developer', System Administrator' and 'Java Application User'.

Description of the change

I have updated the control file for the Ubuntu Software Center VisualVM 'More Info' page.

Changes:
  Changed the bullet point formatting. This was formatted incorrectly and causing the first line to delete the first 2 characters of the word 'VisualVM....'
  I added 's' to the end of the words 'Application Developer', System Administrator' and 'Java Application User'.

To post a comment you must log in.
Revision history for this message
Micah Gersten (micahg) wrote :

@mpt, can you look at this? The adding of the s makes sense, but I'm just wondering if there's any other tweaking needed.

@Paul Stewart
This looks good overall, I'm just asking a colleague with a knack for word interaction to have a look.

Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> @mpt, can you look at this? The adding of the s makes sense, but I'm just
> wondering if there's any other tweaking needed.
>
> @Paul Stewart
> This looks good overall, I'm just asking a colleague with a knack for word
> interaction to have a look.

Thanks Micah,

Please change anything I did as you see fit. I haven't encountered that type of formatting problem before, so I changed it the way I thought it would work. I hope it was helpful.

Thanks
Paul Stewart

Revision history for this message
Matthew Paul Thomas (mpt) :
review: Approve (design)
Revision history for this message
Luke Yelavich (themuso) wrote :

I'd upload this, however I cannot get it to build, due to broken dependencies. The work itself is fine, and the work you have done does not affect the package build process itself, but dependencies further up the chain are currently broken and fail to build, and I have no idea how to properly fix them.

Thanks for your work nonetheless.

If another pilot/sponsor who knos more about java could try and aaddress this properly, that would be appreciated.

review: Approve
Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> I'd upload this, however I cannot get it to build, due to broken dependencies.
> The work itself is fine, and the work you have done does not affect the
> package build process itself, but dependencies further up the chain are
> currently broken and fail to build, and I have no idea how to properly fix
> them.
>
> Thanks for your work nonetheless.
>
> If another pilot/sponsor who knos more about java could try and aaddress this
> properly, that would be appreciated.

Ok, thanks Luke,
Hope it gets worked out ok. Trust I helped a little.

Thanks for your hard work.
Paul Stewart

Revision history for this message
Stefano Rivera (stefanor) wrote :

You need the " ." to separate the paragraphs.

16. By Paul Stewart

Added the "." to separate the paragraphs.

Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> You need the " ." to separate the paragraphs.

Hi Stefano,

I have added the "." to separate the paragraphs as you explained.

Trust this helps. Let me know if you need anything else.

Thanks
Paul

review: Needs Resubmitting
Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> @mpt, can you look at this? The adding of the s makes sense, but I'm just
> wondering if there's any other tweaking needed.
>
> @Paul Stewart
> This looks good overall, I'm just asking a colleague with a knack for word
> interaction to have a look.

Hi Micah,

I had to resubmit it with the '.' to format the paragraphs.
If you have time could you please review it and approve it again?

Thanks
Paul

review: Needs Resubmitting
Revision history for this message
Jani Monoses (jani) wrote :

This could use a changelog entry too, care to craft one? I wonder if we should just upload it anyway to not loose the change and deal with the build breakage later on. Seems like quite some work, try
apt-get build-dep visualvm
Fixing that would be a great new contribution :)

Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> This could use a changelog entry too, care to craft one? I wonder if we should
> just upload it anyway to not loose the change and deal with the build breakage
> later on. Seems like quite some work, try
> apt-get build-dep visualvm
> Fixing that would be a great new contribution :)

Hi Janis,

I'll try to do that.
Thanks
Paul Stewart

Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> This could use a changelog entry too, care to craft one? I wonder if we should
> just upload it anyway to not loose the change and deal with the build breakage
> later on. Seems like quite some work, try
> apt-get build-dep visualvm
> Fixing that would be a great new contribution :)

Hi Janis,

I jumped too soon...I just realized that I actually don't know how to do that. I'll research and see if I can figure it out (it's not too hard is it?). But you may want to load the changes. Your choice of course.

Thanks
Paul

Revision history for this message
Stefano Rivera (stefanor) wrote :

This was merged in visualvm 1.3.2-0ubuntu1

Revision history for this message
Paul Stewart (paulbrianstewart) wrote :

> This was merged in visualvm 1.3.2-0ubuntu1
Cool..thanks

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2010-08-26 22:24:21 +0000
3+++ debian/control 2011-08-18 21:00:30 +0000
4@@ -14,11 +14,11 @@
5 and lightweight profiling capabilities. Designed for both production
6 and development time use, it further enhances the capability of
7 monitoring and performance analysis for the Java SE platform.
8- .
9+.
10 VisualVM is designed for
11- - Application Developer: Monitor, profile, take thread dumps,
12+ - Application Developers: Monitor, profile, take thread dumps,
13 browse heap dumps.
14- - System Administrator: Monitor and control Java applications
15+ - System Administrators: Monitor and control Java applications
16 across the entire network.
17- - Java Application User: Create bug reports containing all the
18+ - Java Application Users: Create bug reports containing all the
19 necessary information.

Subscribers

People subscribed via source and target branches

to all changes: