[ 
https://issues.apache.org/jira/browse/CASSANDRA-5816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13722847#comment-13722847
 ] 

Blair Zajac commented on CASSANDRA-5816:
----------------------------------------

A few points on closing this ticket:

1) I'm new to contributing to the Cassandra eco-system, Java-driver, etc, so I 
would appreciate a one line change instead of just closing a ticket which is 
good at pissing people off (I don't think there's anything more upsetting then 
closing tickets that don't need to be closed), especially new contributors that 
want to make a difference.  Make it so that contributions get into the project 
will have people continue to contribute, even if the change is simple.  I'm a 
MacPorts packager and not every change somebody requests is something I need, 
but I make the change to the package to make it easier for other people [1].

2) This change makes it easier to deploy Cassandra than have to deal with a 
one-off change to have apt ignore recommendations.

3) In 7 years of deploying Ubuntu systems I haven't needed to learn about 
changing recommendations, so make the package work with the standards that 
other packages do.

Thanks,
Blair

[1] https://trac.macports.org/ticket/39793

                
> [PATCH] Debian packaging: also recommend chrony and ptpd in addition to ntp
> ---------------------------------------------------------------------------
>
>                 Key: CASSANDRA-5816
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-5816
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Packaging
>    Affects Versions: 1.2.7
>            Reporter: Blair Zajac
>            Assignee: Blair Zajac
>            Priority: Minor
>
> I'm switching my Ubuntu servers running Cassandra from ntp to chrony for the 
> reasons cited here when Fedora made the switch to have chrony be the default 
> NTP client:
> http://fedoraproject.org/wiki/Features/ChronyDefaultNTP
> Currently, the debian packaging recommends only ntp so if chrony is installed 
> it'll want to remove it and install ntp.  I also added ptpd, the Precision 
> Time Protocol daemon, which is another time syncing server for completeness.
> Please apply this to the 1.2 branch so the next 1.2.x release can deploy with 
> chrony.
> Below is the patch since it's a one-liner.
> Thanks,
> Blair
> --- a/debian/control
> +++ b/debian/control
> @@ -12,7 +12,7 @@ Standards-Version: 3.8.3
>  Package: cassandra
>  Architecture: all
>  Depends: openjdk-6-jre-headless (>= 6b11) | java6-runtime, jsvc (>= 1.0), 
> libcommons-daemon-java (>= 1.0), adduser, libjna-java, python (>= 2.5), 
> python-support (>= 0.90.0), ${misc:Depends}
> -Recommends: ntp
> +Recommends: chrony | ntp | ptpd
>  Conflicts: apache-cassandra1
>  Replaces: apache-cassandra1
>  Description: distributed storage system for structured data

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to