Bug 580830

Technical note added. If any revisions are required, please edit the "Technical Notes" field
accordingly. All revisions will be proofread by the Engineering Content Services team.

New Contents:
A modular framework for implementing a CISCO-like CLI on a *NIX system. Arbitary command menus and actions can be defined using XML files. This software handles the user interaction, and forks the appropriate system commands to perform any actions.

Some comments:

# Summary:        Command Line Interface
That's seems a bit short.

# Group:         Applications
Wrong, might be System Environment/Shells

# License:       GPL 
Wrong, must specify version GPL v1, v2 or v3?

# URL:            
Please fill in.

# Source0:  	http://sourceforge.net/projects/clish/files/clish/0.7.3/%{name}-%{version}.tar.gz   
Wrong, ref: https://fedoraproject.org/wiki/Packaging:SourceURL#Sourceforge.net

# BuildRequires:  autoconf
# BuildRequires:  automake
# BuildRequires:  libtool

Is those needed?

# %description
# 			Command Line Interface
# A modular framework for implementing a CISCO-like CLI on a *NIX system. 
# Arbitary command menus and actions can be defined using XML files. This
#  software handles the user interaction, and forks the appropriate system 
# commands to perform any actions. 
# 

Too long line, keep < 80 chars.

# %install
# rm -rf $RPM_BUILD_ROOT
# make install DESTDIR=$RPM_BUILD_ROOT

Add INSTALL="install -p" to preserve timestamps

# install -d ${RPM_BUILD_ROOT}%{_sysconfdir}/clish
# install -m 775 xml-examples/* ${RPM_BUILD_ROOT}%{_sysconfdir}/clish/

ditto.

# %files
# %defattr(644,root,root,755)

use %defattr(-,root,root,-)

# %doc 

Empty?

# %attr(755,root,root) %{_libdir}/lib*.so.*.*
# %attr(755,root,root) %{_bindir}/*
# %attr(755,root,root) %{_libdir}/lib*.so
# %attr(755,root,root) %{_libdir}/lib*.so.*

Remove %attr and list files a bit more explicit.
 
# %{_libdir}/lib*.la

Don't include la files, remove in install

# %{_includedir}/clish/*.h
# %{_includedir}/lub/*.h
# %{_includedir}/tinyrl/*.h
# %{_includedir}/tinyxml/*.h

Move to devel subpackage

# %{_libdir}/lib*.a
Don't include a files, remove in install

%{_sysconfdir}/clish/*.xml
%{_sysconfdir}/clish/*.txt
%{_sysconfdir}/clish/README
%{_sysconfdir}/clish/*.am

I would prefer only *.xml here 
(real config files in /etc, for documentation use %doc)

# %define date    %(echo `LC_ALL="C" date +"%a %b %d %Y"`)

A new one, please remove and use the common pattern.

# %changelog
# * %{date}   Devil Wang <wxjeacen  at fedoraproject.org>
# - Initial Introduction	<wxjeacen AT,gmail.com>

Not according to guidelines.

Spec URL: <https://fedoraproject.org/w/uploads/e/e1/Clish.spec>
SRPM URL: <https://fedoraproject.org/wiki/File:Clish-0.7.3-1.fc13.src.rpm>

I have updated the SPEC and  SRC-RPM files .

and hope it works.

you can verify the package according to the urls marked before.

Thanks & Regards

Devil_Wang

Okay, as you are not a packager yet, I couldn't do the whole process for getting you to be a Fedora Packager. All I am doing as follows are prereview to get your package closer to be approved:

(In reply to comment #3)
> Spec URL: <https://fedoraproject.org/w/uploads/e/e1/Clish.spec>

* Why your description is shorter than the Summary? Description should be more detail instead by common sense (or at least same amount of content).

* At %configure, have you put --prefix=/usr ? (that should be a macro which could help).

# %description
#    Command Line Interface
# A modular framework for implementing a CISCO-like CLI on a *NIX system. 
# Arbitary command menus and actions can be defined using XML files. This
#  software handles the user interaction, and forks the appropriate system 
# commands to perform any actions. 
# 

Tejie: Too long line, keep < 80 chars.

What Terje said was "the width of each line should be less than 80 chars, but not just cut off the contents from the 2nd line.

In my opinion, the prefix /usr/ is the default parameter for configure .

so , there is no need to add this parameter after configure.

And , I will update the SPEC file for clish package as soon as quickly in session of Describe to add more info about this software.

The use of %configure is correct, no need to add --prefix.

The updated spec looks better, however please add a proper changelog so it's
possible to follow what you have done.

Cause the lasted SPEC did not modify any source code or patches , We can assume the lasted SPEC as the origin SPEC.

So , there is no use to update the changelog for further.

One more question , if my package could be pushed in to the yum list or some other source list for further checking now?

(In reply to comment #8)
> Cause the lasted SPEC did not modify any source code or patches , We can assume
> the lasted SPEC as the origin SPEC.
> 
> So , there is no use to update the changelog for further.

Yes there is. Any time you change something, even the SPEC file, you must bump the release number and update the changelog. Otherwise it is very difficult to follow what you are doing.

* http://fedoraproject.org/wiki/Packaging/UnownedDirectories

* The explicit "ldconfig" dependency for the -devel package is not needed and not true:
>
> Requires: %{name} = %{version}-%{release}, ldconfig

* The package descriptions are very poor:
>
> %description
> Command Line Interface. 

> %description devel 
> Develop Libraries and head files for Clish

update for new SPEC file and new srpm file


Spec URL:
<http://dl.dropbox.com/u/6190834/Clish%20Package/clish.spec>

SRPM URL: 
<https://fedoraproject.org/wiki/File:Clish-0.7.3-2.fc13.src.rpm>

The package fails to build.  Please clear the whiteboard if providing a package which builds.

/bin/sh ./libtool --tag=CXX --mode=link g++  -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic   -o bin/clish  bin/clish.o libclish.la -ltinyrl -ltinyxml -llub -lpthread  
/usr/bin/ld: cannot find -ltinyrl
/usr/bin/ld: cannot find -ltinyxml
/usr/bin/ld: cannot find -llub
collect2: ld returned 1 exit status
make[1]: *** [bin/tclish] Error 1

You should always do either a local mock build or a koji scratch build of every package you submit.  Here's a failing scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2685087

In addition, please do not attach source packages to the wiki; it's really not a general file transfer service.