profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/scottchiefbaker/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

dstat-real/dstat 1282

Versatile resource statistics tool (the real one, not the Red Hat clone)

scottchiefbaker/ESP-WebOTA 211

Simple web based Over-the-Air (OTA) updates for ESP based projects

mmucklo/krumo 75

krumo repository (forked, with updates / new features)

scottchiefbaker/dool 70

Python3 compatible clone of dstat

foglcz/JSONRpc2 20

Generic JSON-RPC v2 implementation

scottchiefbaker/EasyCapture 3

EasyCapture is a simple PHP image hosting and capturing script to make hosting images easier

scottchiefbaker/highlight 2

A command-line tool to filter text and highlight the output.

scottchiefbaker/krumo 2

Krumo with new and updated features

scottchiefbaker/Arduino-Makefile 1

Arduino Makefile to simplify compiling code at the CLI

scottchiefbaker/diff-so-fancy 1

Good-lookin' diffs with diff-highlight and more

pull request commentso-fancy/diff-so-fancy

Trim down npm tarball size

Can this either be updated or closed. Packagers should use the fatpacked version, which is currently 39k.

vladimyr

comment created time in a minute

pull request commentso-fancy/diff-so-fancy

test: use isolated, temporary `GIT_CONFIG`

@rwe false alarm I guess. My local branch had failing tests. When I did a clean checkout all the tests pass. I must have messed up my local branch somehow. We're good.

Nothing to see here. Move along :)

rwe

comment created time in 3 minutes

push eventso-fancy/diff-so-fancy

Scott Baker

commit sha 1a0ad54d77635c6c8a2b994c3df8cde2db82748c

Bump the version so it's clear we're working on new stuff

view details

push time in 4 minutes

pull request commentso-fancy/diff-so-fancy

test: use isolated, temporary `GIT_CONFIG`

Well... maybe I spoke too soon? I merge it, and now some tests are failing?

rwe

comment created time in 5 hours

push eventscottchiefbaker/diff-so-fancy

Robert Estelle

commit sha 700af6e075d2a05f59b16b12fdf6464efd79d880

test: load imports/setup in setup/setup_file bat callbacks

view details

Robert Estelle

commit sha 7fb648e8fa81528455ddc7c11212e605dae3926b

test: cache initial setup vars

view details

Robert Estelle

commit sha f156545804e49f4b604e71e277356ff3c6a8bcd7

test: use completely isolated, temporary GIT_CONFIG This writes a temp git config file during setup/teardown, and exports GIT_CONFIG=… to that and sets GIT_CONFIG_NOSYSTEM=1. This ensures that any git invocations _only_ use that configuration. Note that none of the tests currently test anything other than the default config; however, if they do, the generation of this test config should be moved into `setup()` rather than `setup_file()` so that it can be modified and restored between tests.

view details

Robert Estelle

commit sha fc8e7eed168de4ebb3730e193b7b164782db8b80

fix: over-permissive regex for added/deleted file meta header The previous pattern uselessly included `.*`, unlike all the other patterns, and so would erroneously match on actual diff content lines. Noticed this because the line itself was filtered out of diffs in this codebase. This also tightens the pattern since the file mode will always be six octal (0-7) chars; and in the git codebase the format string is: "%s%snew file mode %06o%s\n"

view details

Scott Baker

commit sha b1deaee002d30e338ba0973255dcb6c259467abb

Merge pull request #421 from rwe/fix-file-meta-regex fix: over-permissive regex for added/deleted file meta header

view details

Scott Baker

commit sha f0672d795bcda5ecc62f88f12bf7c1bc81963c29

Merge branch 'next' of github.com:so-fancy/diff-so-fancy into next

view details

Scott Baker

commit sha 3c4fb51536deef3e226ae3bab00dbc432fb7ea92

Merge pull request #420 from rwe/quarantest test: use isolated, temporary `GIT_CONFIG`

view details

Scott Baker

commit sha 0405628ed5cd4039f30dd97c1cb38fac5f19e5c3

Merge branch 'next' of github.com:so-fancy/diff-so-fancy into next

view details

push time in 5 hours

push eventso-fancy/diff-so-fancy

Robert Estelle

commit sha 700af6e075d2a05f59b16b12fdf6464efd79d880

test: load imports/setup in setup/setup_file bat callbacks

view details

Robert Estelle

commit sha 7fb648e8fa81528455ddc7c11212e605dae3926b

test: cache initial setup vars

view details

Robert Estelle

commit sha f156545804e49f4b604e71e277356ff3c6a8bcd7

test: use completely isolated, temporary GIT_CONFIG This writes a temp git config file during setup/teardown, and exports GIT_CONFIG=… to that and sets GIT_CONFIG_NOSYSTEM=1. This ensures that any git invocations _only_ use that configuration. Note that none of the tests currently test anything other than the default config; however, if they do, the generation of this test config should be moved into `setup()` rather than `setup_file()` so that it can be modified and restored between tests.

view details

Scott Baker

commit sha 3c4fb51536deef3e226ae3bab00dbc432fb7ea92

Merge pull request #420 from rwe/quarantest test: use isolated, temporary `GIT_CONFIG`

view details

push time in 5 hours

pull request commentso-fancy/diff-so-fancy

test: use isolated, temporary `GIT_CONFIG`

This looks cool. I've been wanting to implement something like this for a while, but never had the time. I don't totally understand all of it, so if something breaks down the line (the tests all pass right now) I may have to poke you to help fix things.

Great work.

rwe

comment created time in 5 hours

pull request commentso-fancy/diff-so-fancy

fix: over-permissive regex for added/deleted file meta header

Excellent thanks for the simple concise diff. Makes my job a lot easier.

rwe

comment created time in 5 hours

push eventso-fancy/diff-so-fancy

Robert Estelle

commit sha fc8e7eed168de4ebb3730e193b7b164782db8b80

fix: over-permissive regex for added/deleted file meta header The previous pattern uselessly included `.*`, unlike all the other patterns, and so would erroneously match on actual diff content lines. Noticed this because the line itself was filtered out of diffs in this codebase. This also tightens the pattern since the file mode will always be six octal (0-7) chars; and in the git codebase the format string is: "%s%snew file mode %06o%s\n"

view details

Scott Baker

commit sha b1deaee002d30e338ba0973255dcb6c259467abb

Merge pull request #421 from rwe/fix-file-meta-regex fix: over-permissive regex for added/deleted file meta header

view details

push time in 5 hours

PR merged so-fancy/diff-so-fancy

fix: over-permissive regex for added/deleted file meta header

Noticed this because the line itself was filtered out of diffs in this codebase.

The previous pattern uselessly included .*, unlike all the other patterns, and so would erroneously match on actual diff content lines.

This also tightens the pattern since the file mode will always be six octal (0-7) chars; and in the git codebase the format string is: "%s%snew file mode %06o%s\n".

+2 -2

3 comments

1 changed file

rwe

pr closed time in 5 hours

push eventscottchiefbaker/diff-so-fancy

Paul Irish

commit sha 968214280941a3d8f8004199ef8b9ed0dade5a93

update lockfile

view details

Paul Irish

commit sha 93f60a0765c6c24a030b7a5ccce46743e7483cd6

1.4.2

view details

Paul Irish

commit sha 2719b7a39bdac7c511654de1245f1a103ffbf2bd

use fatpack build for npm

view details

Paul Irish

commit sha a673cb4d2707f64d92b86498a2f5f71c8e2643d5

1.4.3

view details

Scott Baker

commit sha 1a1445cd166cb35b09ff2f558317188d16df3498

Merge pull request #417 from so-fancy/npmusefatpack use fatpack build for npm

view details

Robert Estelle

commit sha 5d567c175dfd40336609b11b27a2049d191d4912

regex: factor out single-sequence ansi_regex

view details

Robert Estelle

commit sha c60e02c4d8dd0c575d8555fba6b535f18649fc40

fix: incorrect starts_with_ansi check for forced coloring

view details

Robert Estelle

commit sha 384ec32c53f452a33fd191bbe5e357d39b31ad5e

fix: starts_with_ansi always returned 1

view details

Scott Baker

commit sha c3205c8a2633a311110fa8609f97b22388298ac8

Merge pull request #422 from rwe/fix-start-with-ansi fix: `starts_with_ansi` always returned 1 and had inverted logic

view details

Scott Baker

commit sha 4def2f885e4bc0a878052a055915601194bd2859

Merge branch 'master' of github.com:so-fancy/diff-so-fancy into next

view details

push time in 2 days

pull request commentso-fancy/diff-so-fancy

fix: over-permissive regex for added/deleted file meta header

This PR also looks good. I just noticed that last PR was against master.

Per our contribution guidelines, all PRs should be against next. If you redo this against next I'll merge it.

rwe

comment created time in 2 days

pull request commentso-fancy/diff-so-fancy

fix: `starts_with_ansi` always returned 1 and had inverted logic

Interesting... good catch!

rwe

comment created time in 2 days

push eventso-fancy/diff-so-fancy

Robert Estelle

commit sha 5d567c175dfd40336609b11b27a2049d191d4912

regex: factor out single-sequence ansi_regex

view details

Robert Estelle

commit sha c60e02c4d8dd0c575d8555fba6b535f18649fc40

fix: incorrect starts_with_ansi check for forced coloring

view details

Robert Estelle

commit sha 384ec32c53f452a33fd191bbe5e357d39b31ad5e

fix: starts_with_ansi always returned 1

view details

Scott Baker

commit sha c3205c8a2633a311110fa8609f97b22388298ac8

Merge pull request #422 from rwe/fix-start-with-ansi fix: `starts_with_ansi` always returned 1 and had inverted logic

view details

push time in 2 days

PR merged so-fancy/diff-so-fancy

fix: `starts_with_ansi` always returned 1 and had inverted logic

$ansi_color_regex is defined as (…)? and so /^$ansi_color_regex/ will always match. This PR factors the ANSI-sequence part of the regex itself and checks (non-optionally) against that.

Additionally, the logic checking starts_with_ansi was inverted, likely to compensate for the above error, but as a side effect forcing diff-so-fancy always to color lines unnecessarily.

+5 -3

0 comment

1 changed file

rwe

pr closed time in 2 days

push eventscottchiefbaker/perl5

Dan Book

commit sha c2ad6dafc59d8d9369dce3e7a16dfe04c43872ef

perlop - clarify that hyphens are interpreted literally in tr with single quotes

view details

Sergey Poznyakoff

commit sha 2fb69afd11f0d2b487eaeb5b057e20b86053fd34

Fix Memoize tests, where GDBM_File is involved * cpan/Memoize/t/errors.t: Use GDBM_NEWDB as the flags argument when tying to GDBM_File. * cpan/Memoize/t/tie_gdbm.t: Likewise.

view details

Tony Cook

commit sha 7b1885ae53fad4123864416d9ef0bbdb78a1882f

add customized entries for the latest Memoize changes

view details

push time in 5 days

push eventscottchiefbaker/perl5

Scott Baker

commit sha 818defb9eb745d8ae4b08e9f313bde316f68aced

Simply the uc() and lc() documentation for novices

view details

Tony Cook

commit sha a5823872283be23731f1bcde7e19a926c44b31a4

prevent undefined behaviour at a language level while probing getenv This test in Configure tries to probe for undefined behaviour in getenv(), but provokes undefined behaviour in C/C++ by falling off the end of a function with a non-void return type. Without optimization clang++ generated a ud2 instruction here on amd64 producing an illegal instruction exception. With optimization the test case fell off the end and started re-executing main(), eventually producing a SIGBUS. Simply dropping the value of getenv() here and returning NULL wasn't useful, under -O2 the compiler optimized away the getenv() call, voiding the whole point of the test.

view details

Tony Cook

commit sha 8d8a6cf5e433f410d4e17f0c9028364384f5915b

remove code that assuming finding pwd on the path is reasonable We deliberately clear PATH when invoking pwd, so this search is useless.

view details

Tony Cook

commit sha a52672c51add571c587f2632221f830dc9d70375

don't fallback to simple pwd When _backtick_pwd invokes $pwd_cmd it first clears the PATH, and since the command has no shell metacharacters, it perl won't invoke the shell, so it will always fail. An alternative here might be to use "/bin/sh -c pwd" but there's no guarantee that pwd is available as a shell builtin.

view details

Tony Cook

commit sha 44e4059baf939468ae61cb63c0c2871c40147dc3

avoid a prototype warning assigning \&getcwd to *cwd This would produce a warning if we fallback to using getcwd() where getcwd() has a prototype.

view details

Tomasz Konojacki

commit sha 6e512bc2beef5b8b17502db7edf50b5984776318

replace all instances of PERL_IMPLICIT_CONTEXT with MULTIPLICITY Since the removal of PERL_OBJECT (acfe0abcedaf592fb4b9cb69ce3468308ae99d91) PERL_IMPLICIT_CONTEXT and MULTIPLICITY have been synonymous and they're being used interchangeably. To simplify the code, this commit replaces all instances of PERL_IMPLICIT_CONTEXT with MULTIPLICITY. PERL_IMPLICIT_CONTEXT will stay defined for compatibility with XS modules.

view details

James E Keenan

commit sha f6111fa5f05db77b7dc3d0597790cafcc7763d98

Skip tests of CORE::Dump on FreeBSD Because on this OS this test writes to /var/log/messages For: https://github.com/Perl/perl5/issues/18847 Move FreeBSD skip condition near to other 'skip_all' conditions, per review by TonyC.

view details

Dave Lambley

commit sha 7d4b8f5eb75f985fe5514ff9436ae487d11c0c2d

Add missing quotes to IO::Socket::INET example Also clean trailing white space. Committer: Dave Lambley is now a Perl author.

view details

Nicholas Clark

commit sha 7886147fdc48d029686af31c71c808adad1a5737

Add -Werror=vla by default. C99 variable-length arrays are dangerous. C11 makes them optional, and MSVC doesn't support them. We can't use them (but even if we could, we should not).

view details

Karl Williamson

commit sha 1a7378185127545dc457ecb21d5ba5174399c6df

regcomp.h: Clarify comment

view details

Karl Williamson

commit sha 19c4b24fc62a1e8dd1f7dcaaf337d0eb11dbb6b0

regcomp.c: Fix typo in comment

view details

Karl Williamson

commit sha bfe5127121a2046fe9d996a64635f6f08d940528

Configure changes for HAS_NON_INT_BITFIELDS This probe has been available without my realizing it had been merged. Some compilers, chiefly IBM, use only integer-size bitfields, warning when presented with other-sized ones. Allowing other-sized fields is an extension to the C standard. These warnings don't affect the correctness of the code generated, but very many are generated per run, potentially overwhelming the reader into not noticing warnings that are important.

view details

Karl Williamson

commit sha 7db2b62c4fe1c3f09543a7b0141a489dd7577911

Configure: apply recent metaconfig changes Several new probes have been added.

view details

Max Maischein

commit sha d94230fbca6fd7ee27db9b233b9188a235732405

Update Socket from CPAN to 2.032

view details

James E Keenan

commit sha 2f19223f3913107e91c5fc2846839677a6398e27

perldelta entry for 1c1bafd (Unicode-Collate upgrade)

view details

James E Keenan

commit sha bc51a73354824f9695c8d108943911bccc6d6c50

perldelta entry for 52f0602 (IO upgrade)

view details

Ricardo Signes

commit sha 60ab4d25a67394c5b528c239e3dc48ce41bc4f91

CPAN mirror list: stop building one

view details

Ricardo Signes

commit sha ee64eb7d2d52cccd588a0f21c548481e56b2e469

use www.cpan.org, not ftp.funet.fi, to check the CPAN

view details

Ricardo Signes

commit sha d6491d6b79de501cf3ab8a7ef8a9767dc9fd108d

perlgov: re-alphabetize the name list

view details

Karl Williamson

commit sha 291b2a970a1a3a8571cae4b8a1f414e7966399ef

regcomp.c: White-space only My attempt to insulate from the leading tab removal the year-old commits finally pushed as 77a6d54c0deb1165b37dcf11c21cd334ae2579bb and 403d7eb3e4320188571cf61b9dab62ff10799f49 failed miserably. I spent a bunch of time sorting it all out, and this is the result.

view details

push time in 6 days

issue commentBenMenking/routeros-api

disabled/enable NAT

I think you want something like this:

$cmd    = "/ip/firewall/nat/set";
$params = ['.id' => 3, 'disabled' => 'yes'];
$data   = $API->comm($cmd, $params);
abdkaviani

comment created time in 14 days

push eventscottchiefbaker/php-Easy-Todo

Scott Baker

commit sha 28b487abbb372ed8f90602175db7d67def14a73e

E-NOTICE fixes

view details

push time in a month

issue commentbenschweizer/iops

Python3 compatibility

It's 2021 now... and Python 2 on modern distros can be problematic. Any updates on a Python 3 version?

a1exsh

comment created time in a month

issue commentso-fancy/diff-so-fancy

Started getting a stacktrace/core dump using diff-so-fancy in powershell

It really shouldn't be possible for d-s-f to crash/stack trace as it's just a Perl script. At worst it should throw a Perl exception and dye cleanly.

I suspect you have an underlying issue either with your Perl installation, or your Powershell installation. Do other shell things work OK?

theperm

comment created time in a month

issue closedPerl/perl5

[feature] Add a `trim()` function to CORE

Perl is one of the strongest string manipulation languages around. Somehow Perl lacks a trim() function in CORE to remove white space. All modern string based languages have something similar: PHP, Javascript, Ruby, Raku, and Python. Can Perl please have one too?

I propose adding trim() to CORE for the next release of Perl.

Relevant Reddit converstion on the topic.

closed time in a month

scottchiefbaker

PR closed Perl/perl5

Reviewers
Request to add a trim() function to Core awaiting review hasConflicts

This should address #17952

Many thanks to Grinnz and LeoNerd for all their help on this pet peeve of mine.

+359 -74

84 comments

26 changed files

scottchiefbaker

pr closed time in a month

pull request commentsmarty-php/smarty

Feature/php8 support

Can we get a pre-release or release candidate of version 4.0? I know there are a lot of people who would be interested in testing things against PHP 8.x.

wisskid

comment created time in a month

issue openedtasmota/tasmotizer

Feature Request: Add support to flash ESP32 devices

Currently the ESP32 requires flashing Tasmota with three different .bin files. It would be really cool if Tasmotizer had support for ESP32 so I can flash my chips easier than a long/complex esptool.py command.

created time in 2 months

push eventscottchiefbaker/Arduino-Kodi-Elapsed

Scott Baker

commit sha f766112669e9be77e1597159b52727f9ae08a117

IP should be localhost

view details

push time in 2 months

push eventscottchiefbaker/Arduino-Kodi-Elapsed

Scott Baker

commit sha a703025df1a22652891fc95d9488704fca7a8a59

Make the sleep cycle exactly 0.5 seconds so the updates are more accurate

view details

push time in 2 months

push eventscottchiefbaker/trob

Scott Baker

commit sha 181175c0a35b9027257ee6e8faca3e07b5773d7e

Include the smarty version in the debug output

view details

push time in 2 months

pull request commentsmarty-php/smarty

Feature/php8 support

I'm testing this on my site now, and so far things look good. Thanks for updating!

wisskid

comment created time in 2 months

issue commentsmarty-php/smarty

"Key not defined" compile time related issue with sysplugins/smarty_internal_debug.php

Using the branch from #629 the follow silenced the above error I mentioned.

$smarty->muteUndefinedOrNullWarnings(true)

It's worth noting that you can also do:

$smarty->error_reporting = E_ALL & ~E_NOTICE & ~E_WARNING

On Smarty 3.x if you just want to get around this on the older smarty until 4.x can be released. It's not an ideal solution, but it works.

ophian

comment created time in 2 months