Bug 242539 - Review Request: freetds - Implementation of the TDS (Tabular DataStream) protocol
Summary: Review Request: freetds - Implementation of the TDS (Tabular DataStream) prot...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-06-04 18:41 UTC by Dmitry Butskoy
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-06-13 16:26:42 UTC
Type: ---
Embargoed:
hdegoede: fedora-review+
wtogami: fedora-cvs+


Attachments (Terms of Use)

Description Dmitry Butskoy 2007-06-04 18:41:31 UTC
Spec URL: http://dmitry.butskoy.name/freetds/freetds.spec
SRPM URL: http://dmitry.butskoy.name/freetds/freetds-0.64-4.src.rpm

Description: 
FreeTDS is a project to document and implement the TDS (Tabular
DataStream) protocol. TDS is used by Sybase(TM) and Microsoft(TM) for
client to database server communications. FreeTDS includes call
level interfaces for DB-Lib, CT-Lib, and ODBC.

Comment 1 Dmitry Butskoy 2007-06-04 18:46:53 UTC
May be avoid trademarks in the Summary?...


Comment 2 Peter Lemenkov 2007-06-04 19:22:41 UTC
Looks like we can't include FreeTDS into main Fedora's repo:

http://fedoraproject.org/wiki/ForbiddenItems#FreeTDS

You'd better post review request to Livna's bugzilla.

Comment 3 Hans de Goede 2007-06-04 19:40:42 UTC
(In reply to comment #1)
> May be avoid trademarks in the Summary?...
> 


Yes that would be a good idea.

(In reply to comment #2)
> Looks like we can't include FreeTDS into main Fedora's repo:
> 
> http://fedoraproject.org/wiki/ForbiddenItems#FreeTDS
> 
> You'd better post review request to Livna's bugzilla.

That info is out of date, see the recent thread on the fedora-devel mailinglist.
I've updated the wiki to match the current state of afairs, this ackage actually
comes from livna and is moving to Fedora, with permission of Tom "Spot" Callaway
(@redhat.com).


Comment 4 Dmitry Butskoy 2007-06-11 13:21:48 UTC
ping Hans... :)


Comment 5 Hans de Goede 2007-06-11 19:48:16 UTC
MUST:
=====
* rpmlint output is:
W: freetds strange-permission freetds.spec 0666
W: freetds strange-permission freetds-0.64-makefile-doc.patch 0666
W: freetds strange-permission freetds-0.64.tar.gz 0666
# You should check these, might be something with my setup though
* Package and spec file named appropriately
* Packaged according to packaging guidelines
* License ok
* spec file is legible and in Am. English.
* Source matches upstream
* Compiles and builds on devel x86_64
* BR: ok
* No locales
* Shared libraries, ldconfig run as required
* Not relocatable
* Package owns / or requires all dirs
* No duplicate files & Permissions ok
* %clean & macro usage OK
* Contains code only
* %doc does not affect runtime, and isn't large enough to warrent a sub package
* -devel package as needed
* no .desktop file required

Should Fix
==========
* This:
cr=`echo -en '\r'`
sed -i "s/$cr//" doc/tds_ssl.html
Is a rather convoluted way of saying this:
sed -i 's/\r//' doc/tds_ssl.html

About the trademarks in the description, I think there is no other sensible way
to describe this package and that the TM use is a typical case of fair use.

You might want to add a sentence: "All trademarks are the property of their
respective owners".

I see no blockers, approved!


Comment 6 Dmitry Butskoy 2007-06-13 13:08:07 UTC
> strange permission 0666
Livna does not play with umask too... :)

> sed -i 's/\r//' doc/tds_ssl.html
Sure, done.

Summary changed to just: "Implementation of the TDS (Tabular DataStream) protocol"

Comment 7 Dmitry Butskoy 2007-06-13 13:10:34 UTC
New Package CVS Request
=======================
Package Name: freetds
Short Description: Implementation of the TDS (Tabular DataStream) protocol
Owners: dmitry
Branches: FC-6 F-7
InitialCC: 

Comment 8 Dmitry Butskoy 2007-06-13 16:26:42 UTC
Build OK, see results at http://koji.fedoraproject.org/packages/freetds/

Comment 9 Dmitry Butskoy 2007-07-25 14:18:47 UTC
Package Change Request
======================
Package Name: freetds
New Branches: EL-4 EL-5


Note You need to log in before you can comment on or make changes to this bug.