[OpenWrt-Devel] [PATCH v3] base-files: make wifi report unknown command
Jonas Gorski
jonas.gorski at gmail.com
Fri Aug 10 04:19:47 EDT 2018
On 9 August 2018 at 20:35, Thibaut <hacks at slashdirt.org> wrote:
>
>> On 9 Aug 2018, at 19:58, Jonas Gorski <jonas.gorski at gmail.com> wrote:
>>
>> On 9 August 2018 at 18:49, Thibaut VARÈNE <hacks at slashdirt.org> wrote:
>>> Avoid having /sbin/wifi silently ignore unknown keywords and execute
>>> "enable"; instead display the help message and exit with an error.
>>>
>>> Spell out the 'enable' keyword and preserve the implicit assumption
>>> that runing /sbin/wifi without argument performs "enable".
>>>
>>> Signed-off-by: Thibaut VARÈNE <hacks at slashdirt.org>
>>> ---
>>
>> Please keep a changelog here to know what's changed between submissions.
>
> it’s a 2-liner patch. I don’t think it warrants a changelog in the commit message :)
The fact that you needed to send 4 versions should be enough of an
argument to keep a changelog regardless of the size of the patch ;)
Also the changelog belongs below the tear-off line ("---") so it
doesn't end in the commit message.
So please keep that in mind for future patches.
Regards
Jonas
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list