[collectd] cpufreq for freebsd patch

Marco van Tol marco at tols.org
Wed Apr 24 11:41:46 CEST 2019


Hi Matthias,

Thanks for your mail, this is very easy.
I have done exactly that, and now it's "all checks passed" :-)

Thanks!

Marco van Tol


> Op 24 apr. 2019, om 09:47 heeft Matthias Runge <mrunge at matthias-runge.de> het volgende geschreven:
> 
> On Tue, Apr 23, 2019 at 11:31:27PM +0200, Marco van Tol wrote:
> Marco,
> 
> the ChangeLog feature asks for adding a description to the commit
> message, see here:
> 
> https://github.com/collectd/collectd/blob/master/docs/CONTRIBUTING.md#changelog
> 
> Another example is here: https://github.com/collectd/collectd/pull/3139
> 
> Hope that helps,
> 
> Matthias
> 
>> Hi Pavel,
>> 
>> I created a PR now, requesting to merge ometol/collectd:master into collectd/collectd:master.
>> 
>> It complains about that I did not update ChangeLog.
>> After looking at that file I can of course add a date and a line to it indicating what I did.
>> Do you want me to do this?
>> 
>> Many thanks in advance!
>> 
>> Marco van Tol
>> 
>> 
>> 
>>> Op 23 apr. 2019, om 19:59 heeft Marco van Tol <marco at tols.org> het volgende geschreven:
>>> 
>>> Hi Pavel,
>>> 
>>> Many thanks for your advise.  I'll do what you suggest soon.
>>> 
>>> Marco van Tol
>>> 
>>> 
>>> 
>>>> Op 23 apr. 2019, om 19:50 heeft Pavel <pavel2000 at ngs.ru> het volgende geschreven:
>>>> 
>>>> 23.04.2019 23:03, Marco van Tol :
>>>> 
>>>> Hi, Marco!
>>>> 
>>>> For your work to be merged into upstream branch, you have to do Pull Request on Github.
>>>> 
>>>> Also, 'src/collectd.conf.pod' file needs to be updated too. Can you please do that?
>>>> "This plugin is available on Linux and FreeBSD only"
>>>> "On Linux, it reads ... " + "On FreeBSD, ... "
>>>> 
>>>> If you do PR on GitHub I will take some care about future steps.
>>>> 
>>>> Thanks for your work on Collectd!
>>>> 
>>>>> Hi there,
>>>>> 
>>>>> This is my first time I do something like a pull request.
>>>>> 
>>>>> Please see attached patch that will allow FreeBSD systems to also enjoy the cpufreq plugin.
>>>>> 
>>>>> The github fork of my work can be found here:
>>>>> 
>>>>> https://github.com/ometol/collectd
>>>>> 
>>>>> 
>>>>> Does this give you enough to consider a merge, or is there anything else I should provide?
>>>>> 
>>>>> Thank you very much in advance!
>>>>> 
>>>>> Marco van Tol
>>>>> 
>>>> 
>>>> 
>>>> 
>>>> 
>>>>> 
>>>>> 
>>>>> _______________________________________________
>>>>> collectd mailing list
>>>>> 
>>>>> collectd at verplant.org
>>>>> https://mailman.verplant.org/listinfo/collectd
>>>> 
>>>> 
>>>> 
>>> 
>>> -- 
>>> Marco van Tol
>>> 
>>> 
>> 
>> -- 
>> Marco van Tol
>> 
>> 
>> _______________________________________________
>> collectd mailing list
>> collectd at verplant.org
>> https://mailman.verplant.org/listinfo/collectd
> 
> -- 
> Matthias Runge <mrunge at matthias-runge.de>
> 
> _______________________________________________
> collectd mailing list
> collectd at verplant.org
> https://mailman.verplant.org/listinfo/collectd

-- 
Marco van Tol




More information about the collectd mailing list