- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jul 19 2018
Jul 16 2018
Jul 9 2018
Jul 8 2018
Jun 6 2018
@kib Updated by tree to r334733 and applied the patch, it's working as expected, thanks!
Apr 20 2018
Feb 11 2018
Fix typo and extra whitespace
In D14103#300013, @emaste wrote:Testing with this change on X260 (LEN0068, MHKV 0x0200 AFAICT) I see warnings like:
# sysctl dev.acpi_ibm.0.kbd_backlight=0 ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383) ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383) dev.acpi_ibm.0.kbd_backlight: 1ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383) -> ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383) ACPI Warning: \_SB.PCI0.LPC.EC.HKEY.MLCG: Insufficient arguments - Caller passed 0, method requires 1 (20180209/nsarguments-383) 0 #Keyboard backlight control works despite the warnings. I haven't tested this machine on an unpatched kernel. echo 1 > /dev/led/micmute has no effect.
Feb 9 2018
Add support for controlling micmute LED
Feb 2 2018
Please note that man page changes are pending, but will be added to the review before committing any changes.
Jan 30 2018
Add support for controlling the keyboard backlight found in newer lenovo models.
Jan 29 2018
@rodrigc I think this can be closed.
Jan 25 2018
Jan 24 2018
Jan 22 2018
Sep 24 2017
@koobs Just glanced at it, trusting your judgement.
Sep 7 2017
Sep 4 2017
Sep 1 2017
Thanks for taking over, I don't have any recent hardware or the time to maintain/test this myself at the moment.
Aug 8 2017
Jul 1 2017
Apr 7 2017
Mar 29 2017
Mar 28 2017
Upstream fixed this in the meantime.
Never mind, apparently I can accept it myself and this is good enough to allow me to close it.
Update to 0.9.5, including missing files and fixing pending upstream.
Use PORTVERSION in dependencies (as all parts of buildbot need
to get updated at the same time.
As the patch gets quite complicated and I would like to reference this review in the commit, I'll take over the revision and update the diff.
@rodrigc This commit breaks most of the unit tests:
In D10162#210112, @mat wrote:You missed the third step which tells you to do "svn move" (or "svn copy", but here it is svn move.)
As it is too late, because buildbot-slave is gone, it is going to be a bit more complicated, a bit of a mix from the copy and the resurection. What you're going to need to do is:
- copy the current buildbot-worker directory somewhere
- svn rm buildbot-worker
- svn commit (this will break INDEX and ports that have a dependency on it, but you're fixing it in the next commit, so, it's "ok")
- svn cp 'svn+ssh://repo.freebsd.org/ports/head/devel/buildbot-slave/@437058' buildbot-worker
- pull back the changes from buildbot-slave to buildbot-worker.
- svn commit (this will unbreak INDEX and all)
In D10162#210106, @mat wrote:Before any work on buildbot-worker can get in the tree, it needs to be removed and added back properly.
Thanks for opening this, much appreciated. I started working on this in parallel and there are a few more changes to it (running unit tests showed more dependencies that are missing - maybe only for the test target, but unclear at this point). Once I figured out all the details I'll commit the updated versions, until then it's probably best if you abandon this review.
Mar 27 2017
Mar 16 2017
Mar 15 2017
Mar 12 2017
Feb 28 2017
Feb 2 2017
Jan 30 2017
Jan 29 2017
Jan 16 2017
Nov 15 2016
Nov 5 2016
Oct 28 2016
Oct 12 2016
Oct 3 2016
Sep 28 2016
@avg Unfortunately I don't have much time at my hands right now and - like you pointed out - this has been committed quite a while ago. So I think it's best if you open a PR and create a patch/new code review. If time permits, I'm more than happy to help you testing the improved code.
Aug 16 2016
Aug 10 2016
Jul 10 2016
Jun 13 2016
Jun 6 2016
Obviously, thanks.
Jun 5 2016
Jun 4 2016
May 30 2016
May 28 2016
Fix comment as pointed out by @jhb
May 27 2016
Apr 6 2016
Apr 2 2016
Add -s to grep to silence error messages in stages of the build
when certain files don't exist yet.
Mar 31 2016
@hrs (edited, so it makes more sense)
@hrs
Can you please suggest how to accomplish what this is doing (and I think what this is doing is extremely important, I was embarrassed when I showed the Porters Handbook to someone and it said that it was last updated two years ago) in a style sheet in a way that is less complex (this is not about parsing XML, but about getting an SVN keyword out of a group of text files, which is super stable, compatible and will work even on files that are part of the documentation, even if they're not XML)?
Mar 27 2016
Kind of went the opposite direction in terms of perl, I find the result to be quite readable though.