[prev in list] [next in list] [prev in thread] [next in thread] 

List:       opensuse-packaging
Subject:    Re: [opensuse-packaging] Review package udev-browse
From:       Peter Linnell <plinnell () scribus ! net>
Date:       2012-04-17 5:00:29
Message-ID: 4F8CF8ED.1040709 () scribus ! net
[Download RAW message or body]

On 04/16/2012 01:55 PM, Greg Freemyer wrote:
> On Mon, Apr 16, 2012 at 2:10 PM, Manu Gupta<manugupt1@gmail.com>  wrote:
> > Hi all
> > 
> > Thanks for the reviews.
> > 
> > On Mon, Apr 16, 2012 at 1:02 PM, Andreas Jaeger<aj@suse.com>  wrote:
> > > On Sunday, April 15, 2012 20:11:36 Manu Gupta wrote:
> > > > Hi All,
> > > > 
> > > > Can anyone review
> > > > https://build.opensuse.org/package/show?package=udev-browse&project=home
> > > > %3Amanugupt1.
> > > 
> > > I suggest to word-wrap the decription after 75 chars. Rest looks fine to
> > > me.
> > > 
> > Sorry, but I did not understand this part.
> > 
> 
> He's talking about your specfile.
> 
> Look at
> 
> https://build.opensuse.org/package/view_file?file=udev-browse.spec&package=udev-browse&project=home%3Amanugupt1&rev=3472e598cdfe4f7928aac27e9cfc1dc3
>  
> The %description section should be readable on that display, but it is
> instead one really long line so you have to scroll to read it.  Add
> some carriage returns. like:
> 
> %description
> It's easy to get lost in /sys and not much fun typing
> long udevadm info command lines all the time. Today,
> when I had enough of that I sat down and spent an hour
> to write a little UI for exploring the udev/sysfs tree:
> dev-browse. I wrote it for my own use, but I am quite
> sure I am not the only one who wants a little bit simpler
> access to the device tree
> 
> Greg

And I have accepted it in Utilities, as well as enabling some newer repos.

Thanks,
Peter
-- 
To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org
To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org


[prev in list] [next in list] [prev in thread] [next in thread]