[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: metacity-setup .debs



On Tue, 2002-07-09 at 11:32, Ian Wienand wrote:
> Hi all,
> 
> I have packaged metacity-setup.
> 
> It is available at 
> 
> http://www.wienand.org/debian/metacity-setup
> 
> and works ok for me (and passes lintian ok).
> 
> These are my first packages so please test them and give any feedback.
> >From what I understand, if I want them to go into debian propper
> someone needs to sponsor them -- if anyone is interested that would be
> nice (my request on the sponsorship mailing list was unanswered).

All these files aren't supposed to be in the diff:
+++ metacity-setup-0.6.1/intltool-extract
+++ metacity-setup-0.6.1/config.guess
+++ metacity-setup-0.6.1/intltool-update
+++ metacity-setup-0.6.1/config.sub
+++ metacity-setup-0.6.1/intltool-merge

--- metacity-setup-0.6.1.orig/debian/dirs
+++ metacity-setup-0.6.1/debian/dirs
@@ -0,0 +1,2 @@
+usr/bin
+usr/sbin

usr/sbin isn't needed.

--- metacity-setup-0.6.1.orig/debian/docs
+++ metacity-setup-0.6.1/debian/docs
@@ -0,0 +1,2 @@
+NEWS
+README
--- metacity-setup-0.6.1.orig/debian/control
+++ metacity-setup-0.6.1/debian/control
@@ -0,0 +1,12 @@
+Source: metacity-setup
+Section: x11
+Priority: optional
+Maintainer: Ian Wienand <ianw@ieee.org>
+Build-Depends: debhelper (>> 3.0.0) , metacity, libgtk2.0-dev, libgnomeui-dev, libglade2-dev, libgconf2-dev (>= 1.1.8), libxml2-dev (>= 2.4.19-4), libgconf2-dev (>= 1.1.8), libpopt-dev

metacity isn't a build dependency, it's a run-time dependency.

+Standards-Version: 3.5.2
+
+Package: metacity-setup
+Architecture: any
+Depends: ${shlibs:Depends}, metacity
+Description: Setup program for metacity
+ Provides a graphical interface setup for metacity

Read some other descriptions, it's not really the style of the house:

Metacity-setup is a tool to setup the Metacity window manager via a
graphical interface. It allows you to setup it:
 * Appearance
 * Fonts
 * Focus mode
 * Number of workspaces

--- metacity-setup-0.6.1.orig/debian/rules
+++ metacity-setup-0.6.1/debian/rules

Remove all the unused comments and the ones from the original dh_make run.

--- metacity-setup-0.6.1.orig/debian/changelog
+++ metacity-setup-0.6.1/debian/changelog

OK

--- metacity-setup-0.6.1.orig/debian/copyright
+++ metacity-setup-0.6.1/debian/copyright
@@ -0,0 +1,15 @@
+This package was debianized by Ian Wienand <ianw@ieee.org> on
+Wed,  3 Jul 2002 22:41:28 +1000.
+
+It was downloaded from http://plastercast.tzo.com/~plastercast/Projects/
+
+Upstream Author(s): Peter Snyder (plastercast@plastercast.tzo.com)

There's only one author, so it should be "Author"

--- metacity-setup-0.6.1.orig/debian/metacity-setup.1
+++ metacity-setup-0.6.1/debian/metacity-setup.1
@@ -0,0 +1,32 @@
+.\"                                      Hey, EMACS: -*- nroff -*-
+.\" First parameter, NAME, should be all caps
+.\" Second parameter, SECTION, should be 1-8, maybe w/ subsection
+.\" other parameters are allowed: see man(7), man(1)
+.TH METACITY-SETUP SECTION "July  3, 2002"
+.\" Please adjust this date whenever revising the manpage.
+.\"
+.\" Some roff macros, for reference:
+.\" .nh        disable hyphenation
+.\" .hy        enable hyphenation
+.\" .ad l      left justify
+.\" .ad b      justify to both left and right margins
+.\" .nf        disable filling
+.\" .fi        enable filling
+.\" .br        insert line break
+.\" .sp <n>    insert n+1 empty lines
+.\" for manpage-specific macros, see man(7)
+.SH NAME
+metacity-setup \- graphical setup of metacity window manager
+.SH SYNOPSIS
+.B metacity-setup
+.RI [--usage]
+.br
+This manual page documents briefly the
+.B metacity-setup
+command.
+
+\fBmetacity-setup\fP graphically configures the metacity window manager
+
+.SH AUTHOR
+This manual page was written by Ian Wienand <ianw@ieee.org>,
+for the Debian GNU/Linux system (but may be used by others).

Remove the comments from the manpage. And you might want to update it to
look like other man pages.

Hope this helps

-- 
/Bastien Nocera
http://hadess.net

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: