[Pkg-javascript-devel] Bug#705393: [RFR] templates://sipml5/{sipml5-web-phone.templates}

2013-04-22 Thread Justin B Rye
Christian PERRIER wrote:
 Your review should be sent as an answer to this mail.
[...]
  Template: sipml5-web-phone/webserver
  Type: boolean
  Default: true
 -_Description: Automatically configure apache2?
 +_Description: Automatically configure Apache for Sipml5?
[...]  ^^
 We also need to specify for what it is to be configured, in case the
 question comes in the middle of a bunch of other questions.

Agreed, but the upstream name seems to be SipML5 (or perhaps sipML5).
  
Meanwhile in the control file:
  Multi-Arch: foreign
  Depends: ${misc:Depends}
  Recommends: javascript-common
 -Description: JavaScript implementation of a WebRTC SIP video phone
 - HTML5 SIP client entirely written in JavaScript for integration in
 - any web site, incude a blog, CMS or web application.  Requires
 - a modern WebRTC capable web browser and a SIP server/proxy supporting
 +Description: WebRTC SIP video phone - Javascript library

Exactly what I was thinking, except it's JavaScript.

 + Simpl5 is an HTML5 SIP client entirely written in JavaScript for 
 integration in
  ^^^
If only that acronym worked!

I would also change s/in/into/.

 + any web site, such as a blog, CMS or web application. It requires

Add a comma for d-l-e house style:   ^,

 + a WebRTC-capable web browser and a SIP server/proxy supporting
   the SIP over WebSockets transport.
[...]
 WebRTC-capable : use hyphen.

I'm also tempted to call it the SIP-over-WebSockets transport, but
that's probably overdoing it.

[...]
 -Description: Basic SIP video-phone web page based on WebRTC
 +Description: WebRTC SIP video-phone - sample web page

To me that isn't really a big enough hint at the fact that it actually
implements a SIP client.  How about demonstration web page?

Oh, and I notice only this one hyphenates video-phone.  Okay, let's
standardise on that.
-- 
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package
diff -ru sipml5.pristine/debian/control sipml5/debian/control
--- sipml5.pristine/debian/control	2013-02-23 20:39:43.0 +
+++ sipml5/debian/control	2013-04-22 11:15:09.439629322 +0100
@@ -14,11 +14,11 @@
 Multi-Arch: foreign
 Depends: ${misc:Depends}
 Recommends: javascript-common
-Description: JavaScript implementation of a WebRTC SIP video phone
- HTML5 SIP client entirely written in JavaScript for integration in
- any web site, incude a blog, CMS or web application.  Requires
- a modern WebRTC capable web browser and a SIP server/proxy supporting
- the SIP over WebSockets transport.
+Description: WebRTC SIP video-phone - JavaScript library
+ SipML5 is an HTML5 SIP client entirely written in JavaScript for integration
+ into any web site, such as a blog, CMS, or web application. It requires a
+ WebRTC-capable web browser and a SIP server/proxy supporting the SIP over
+ WebSockets transport.
  .
  This package provides the JavaScript library for use in web sites
  or for other packages that require it.
@@ -28,11 +28,11 @@
 Multi-Arch: foreign
 Depends: ${misc:Depends}
 Recommends: libjs-sipml5
-Description: JavaScript implementation of a WebRTC SIP video phone - API docs
- HTML5 SIP client entirely written in JavaScript for integration in
- any web site, incude a blog, CMS or web application.  Requires
- a modern WebRTC capable web browser and a SIP server/proxy supporting
- the SIP over WebSockets transport.
+Description: WebRTC SIP video-phone - API documentation
+ SipML5 is an HTML5 SIP client entirely written in JavaScript for integration
+ into any web site, such as a blog, CMS, or web application. It requires a
+ WebRTC-capable web browser and a SIP server/proxy supporting the SIP over
+ WebSockets transport.
  .
  This package provides the API documentation for the JavaScript library.
 
@@ -41,11 +41,11 @@
 Multi-Arch: foreign
 Depends: ${misc:Depends}, libjs-sipml5, libjs-jquery, apache2 | lighttpd | httpd
 Recommends: repro, resiprocate-turn-server
-Description: Basic SIP video-phone web page based on WebRTC
- HTML5 SIP client entirely written in JavaScript for integration in
- any web site, incude a blog, CMS or web application.  Requires
- a modern WebRTC capable web browser and a SIP server/proxy supporting
- the SIP over WebSockets transport.
+Description: WebRTC SIP video-phone - demonstration web page
+ SipML5 is an HTML5 SIP client entirely written in JavaScript for integration
+ into any web site, such as a blog, CMS, or web application. It requires a
+ WebRTC-capable web browser and a SIP server/proxy supporting the SIP over
+ WebSockets transport.
  .
  This package provides a basic HTML implementation of a phone that
  can be published through a web server such as Apache.
diff -ru sipml5.pristine/debian/sipml5-web-phone.templates sipml5/debian/sipml5-web-phone.templates
--- sipml5.pristine/debian/sipml5-web-phone.templates	2013-04-14 18:33:39.756121203 +0100
+++ 

[Pkg-javascript-devel] Bug#705393: [RFR] templates://sipml5/{sipml5-web-phone.templates}

2013-04-22 Thread Christian PERRIER
Quoting Justin B Rye (justin.byam@gmail.com):

  Your review should be sent as an answer to this mail.


(snip Justin's review)

I just picked all changes suggested by Justin (who said but this is
always what you're doing anyway! ?).



signature.asc
Description: Digital signature
___
Pkg-javascript-devel mailing list
Pkg-javascript-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-javascript-devel