{"push_id":1896954050,"size":18,"distinct_size":0,"ref":"refs/heads/busybox","head":"ec05df13b0f3bc69074909f078f981f417d95c89","before":"69be994de69d794f038f10a3e7a67519b2006581","commits":[{"sha":"008fc9499ae63186589fbc81a5f849a2df652004","author":{"name":"Ron Yorston","email":"f95b2c4da81e0d633af4e66f01d84a3da68d2d9d@pobox.com"},"message":"libbb: remove vdprintf\n\nTwo reasons:\n\n* it's only built for GNU C Library versions < 2 (released 1997)\n\n* it isn't used anywhere in BusyBox\n\nSigned-off-by: Ron Yorston <rmy@pobox.com>\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/008fc9499ae63186589fbc81a5f849a2df652004"},{"sha":"170f93ef1bc49a9e4c68f85f5245416767c1916f","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: \"Undo all redirections\" comment is wrong, delete it\n\nNo code changes.\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/170f93ef1bc49a9e4c68f85f5245416767c1916f"},{"sha":"a732898fdd9748b966da228ee8bbbc148c3c10c9","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: [PARSER] Removed noexpand/length check on eofmark\n\nUpstream comment:\n\n\tDate: Sun, 11 Nov 2007 14:21:23 +0800\n\t[PARSER] Removed noexpand/length check on eofmark\n\n\tOn Tue, Oct 30, 2007 at 04:23:35AM +0000, Oleg Verych wrote:\n\t>\n\t> } 8<<\"\"\n\t> ======================\n\n\tActually this (the empty delim) only works with dash by accident.\n\tI've tried bash and pdksh and they both terminate on the first\n\tempty line which is what you would expect rather than EOF.  The\n\treal Korn shell does something completely different.\n\n\tI've fixed this in dash to conform to bash/pdksh.\n\n\t> In [0] it's stated, that delimiter isn't evaluated (expanded), only\n\t> quoiting must be checked. That if() seems to be completely bogus.\n\n\tOK I agree.  The reason it was there is because the parser would\n\thave already replaced the dollar sign by an internal representation.\n\n\tI've fixed it properly with this patch.\n\n\tTest case:\n\n\t        cat <<- $a\n\t                OK\n\t        $a\n\n\t        cat <<- \"\"\n\t                OK\n\n\t        echo OK\n\n\tOld result:\n\n\t        dash: Syntax error: Illegal eof marker for << redirection\n\t        OK\n\n\t        echo OK\n\n\tNew result:\n\n\t        OK\n\t        OK\n\t        OK\n\nfunction                                             old     new   delta\nparsefname                                           227     152     -75\nreadtoken1                                          2819    2651    -168\n------------------------------------------------------------------------------\n(add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-243)           Total: -243 bytes\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/a732898fdd9748b966da228ee8bbbc148c3c10c9"},{"sha":"0f018b30700989462de0a15b8285206d16170c1f","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: fix handling of empty heredoc EOF marker\n\nfunction                                             old     new   delta\nparse_stream                                        2609    2634     +25\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/0f018b30700989462de0a15b8285206d16170c1f"},{"sha":"469998015f57a552718420a4050d90e5dae6a6fd","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: [PARSER] Add FAKEEOFMARK for expandstr\n\nUpstream commit:\n\n    Date: Thu, 27 Dec 2007 13:54:16 +1100\n    [PARSER] Add FAKEEOFMARK for expandstr\n\n    Previously expandstr used the string \"\" to indicate that it needs to be\n    treated just like a here-doc except that there is no terminator.  However,\n    the string \"\" is in fact a valid here-doc terminator so now that we deal\n    with it correctly expandstr no longer works in the presence of new-lines\n    in the prompt.\n\n    This patch introduces the FAKEEOFMARK macro which does not equal any\n    real EOF marker but is distinct from the NULL pointer which is used to\n    indicate non-here-doc contexts.\n\n    Thanks to Markus Triska for reporting this regression.\n\n    Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>\n\nUnfortunately, I did not find the failing example for this old fix.\n\nI also tweaked the code which was added by this commit:\n\"\nDate: Mon Sep 24 18:30:02 2007 +0000\nash: fix prompt expansion (Natanael Copa <natanael.copa@gmail.com>)\n\"\nsince other parts of code do expect expandstr() to use DQSYNTAX, not PSSYNTAX.\n\nfunction                                             old     new   delta\nparse_stream                                        2609    2634     +25\nsetprompt_if                                         128     133      +5\nread_profile                                          32      37      +5\nevalcommand                                         1334    1339      +5\nexpandstr                                            122     120      -2\n------------------------------------------------------------------------------\n(add/remove: 0/0 grow/shrink: 4/1 up/down: 40/-2)              Total: 38 bytes\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/469998015f57a552718420a4050d90e5dae6a6fd"},{"sha":"1c79aeb6a8cae4cd4013926b97f39305b689d74c","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: [REDIR] Fix popredir on abnormal exit from built-in\n\nUpstream commit:\n\n    Date: Thu, 27 May 2010 15:03:46 +0800\n    [REDIR] Fix popredir on abnormal exit from built-in\n\n    Just like the poplocalvar problem recently fixed, redirections\n    can also be leaked in case of an abnormal exit.  This patch fixes\n    it using the same method as poplocalvar, by storing the previous\n    redirection state and restoring to that point.\n\n    Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/1c79aeb6a8cae4cd4013926b97f39305b689d74c"},{"sha":"5f0a75f24b5afdedf5b67a7f42184ed196e1a5c9","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: if !ENABLE_ASH_EXPAND_PRMT, disable PSSYNTAX code\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/5f0a75f24b5afdedf5b67a7f42184ed196e1a5c9"},{"sha":"d07a15bd1ba99caa9386234cda1e8c83897c7553","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: remove REDIR_SAVEFD2\n\nfunction                                             old     new   delta\nevalcommand                                         1364    1369      +5\nredirect                                            1055    1014     -41\n------------------------------------------------------------------------------\n(add/remove: 0/0 grow/shrink: 1/1 up/down: 5/-41)             Total: -36 bytes\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/d07a15bd1ba99caa9386234cda1e8c83897c7553"},{"sha":"657e9005a9e31e1da094b260abaa8f335e92301f","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: massage redirect code to be slightly more like ash\n\nfunction                                             old     new   delta\nsave_fd_on_redirect                                    -     203    +203\nxdup_CLOEXEC_and_close                                 -      75     +75\nsetup_redirects                                      245     250      +5\nxdup_and_close                                        72       -     -72\nsave_fds_on_redirect                                 221       -    -221\n------------------------------------------------------------------------------\n(add/remove: 2/2 grow/shrink: 1/0 up/down: 283/-293)          Total: -10 bytes\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/657e9005a9e31e1da094b260abaa8f335e92301f"},{"sha":"035486c7500c09616a6c1040d8e70923532a5c2d","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: significant overhaul of redirect saving logic\n\nNew code is similar to what hush is doing.\nMake CLOSED to -1: same as dash.\npopredir() loses \"restore\" parameter: same as dash.\nCOPYFD_RESTORE bit is no longer necessary.\n\nThis change fixes this interactive bug:\n\n\t$ ls -l /proc/$$/fd 10>&-\n\tash: can't set tty process group: Bad file descriptor\n\tash: can't set tty process group: Bad file descriptor\n\t[1]+  Done(2)                    ls -l /proc/${\\$}/fd 10>&4294967295\n\nfunction                                             old     new   delta\nunwindredir                                           29      27      -2\ntryexec                                              154     152      -2\nevaltree                                             503     501      -2\nevalcommand                                         1369    1367      -2\ncmdloop                                              187     185      -2\nredirect                                            1029    1018     -11\npopredir                                             153     123     -30\nneed_to_remember                                      36       -     -36\nis_hidden_fd                                          68       -     -68\n------------------------------------------------------------------------------\n(add/remove: 0/2 grow/shrink: 0/7 up/down: 0/-155)           Total: -155 bytes\n   text    data     bss     dec     hex filename\n 914572     485    6848  921905   e1131 busybox_old\n 914553     485    6848  921886   e111e busybox_unstripped\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/035486c7500c09616a6c1040d8e70923532a5c2d"},{"sha":"32fdf2f9fc9a617918672d71579f4ad42eb9bde9","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash,hush: \">&10\" redirects to script/tty fds should not work\n\nThe fact that shell has open fds to tty and/or scripts should be\nunobservable, if possible. In particular, if redirect tries to dup\none of them via \">&script_fd\", it's better to pretend that script_fd\nis closed, and thus redirect fails with EBADF.\n\nFixes these two testcase failures:\nash-redir/redir_to_bad_fd.tests\nhush-redir/redir_to_bad_fd3.tests\n\nfunction                                             old     new   delta\nredirect                                            1018    1129    +111\nsetup_redirects                                      250     359    +109\nreadtoken1                                          2651    2655      +4\ncmdloop                                              185     187      +2\nchangepath                                           194     195      +1\nsave_fd_on_redirect                                  203     194      -9\nevaltree                                             501     484     -17\n------------------------------------------------------------------------------\n(add/remove: 0/0 grow/shrink: 5/2 up/down: 227/-26)           Total: 201 bytes\n   text    data     bss     dec     hex filename\n 914553     485    6848  921886   e111e busybox_old\n 914754     485    6848  922087   e11e7 busybox_unstripped\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/32fdf2f9fc9a617918672d71579f4ad42eb9bde9"},{"sha":"bf1c344dfdc6f38ad6aa81c10b7b050e0dfc5d96","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: if STANDALONE, close interactive fd for NOEXECed children\n\nfunction                                             old     new   delta\npseudo_exec_argv                                     291     305     +14\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/bf1c344dfdc6f38ad6aa81c10b7b050e0dfc5d96"},{"sha":"75481d363454a3e0640a5bd0f78d2cc4403e6235","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: functions have priority over builtins (!)\n\nfunction                                             old     new   delta\npseudo_exec_argv                                     291     305     +14\nrun_pipe                                            1560    1555      -5\n------------------------------------------------------------------------------\n(add/remove: 0/0 grow/shrink: 1/1 up/down: 14/-5)               Total: 9 bytes\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/75481d363454a3e0640a5bd0f78d2cc4403e6235"},{"sha":"d0fff9155bc55d9576be4c6ba5f656f5b4856ac4","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: fix display of \">&-\" redirect in job strings\n\nfunction                                             old     new   delta\ncmdtxt                                               558     569     +11\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/d0fff9155bc55d9576be4c6ba5f656f5b4856ac4"},{"sha":"5b3d2eb327ce7e1c55c6c94bc70782f733d59990","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: fix \"true | func_with_return\" not allowing return.\n\nfunction                                             old     new   delta\npseudo_exec_argv                                     305     312      +7\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/5b3d2eb327ce7e1c55c6c94bc70782f733d59990"},{"sha":"cee603d921594fc779bc26a200dfb010cd62ab92","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"hush: remove redundant \"G_flag_return_in_progress = -1\"\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/cee603d921594fc779bc26a200dfb010cd62ab92"},{"sha":"78b1b1b07ac2784f67bc65bb1d86cc2559a12446","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"test: fix 4-argument case\n\nUpstream dash commit:\n\n    Date: Sun, 13 Jul 2008 19:20:10 +0800\n    Fixed 3,4-argument cases for test per POSIX\n\n    ----- Forwarded message from Gerrit Pape <pape@smarden.org> -----\n\n    Subject: Bug#455828: dash: 4-argument test \"test \\( ! -e \\)\" yields an error\n    Date: Fri, 28 Dec 2007 08:53:29 +0000\n    From: Gerrit Pape <pape@smarden.org>\n    To: Vincent Lefevre <vincent@vinc17.org>, 455828@bugs.debian.org\n\n    On Thu, Dec 27, 2007 at 06:23:20PM +0100, Vincent Lefevre wrote:\n    > On 2007-12-27 16:00:06 +0000, Gerrit Pape wrote:\n    > > On Wed, Dec 12, 2007 at 02:18:47AM +0100, Vincent Lefevre wrote:\n    > > > According to POSIX[*], \"test \\( ! -e \\)\" is a 4-argument test and is\n    > > > here equivalent to \"test ! -e\". But dash (like ksh93 and bash) yields\n    > > > an error:\n    > > >\n    > > > $ test \\( ! -e \\) || echo $?\n    > > > test: 1: closing paren expected\n    > > > 2\n    > > > $ test ! -e || echo $?\n    > > > 1\n    > >\n    > > Hi Vincent,\n    > >\n    > > the -e switch to test takes an argument, a pathname.\n    >\n    > According to POSIX, in both above examples, \"-e\" is *not* a switch,\n    > just a string.\n    >\n    >   test \\( ! -e \\)\n    >\n    > means: return true if the string \"-e\" is empty, otherwhise return false.\n    > The error in dash is that it incorrectly thinks that \"-e\" is a switch in\n    > this context.\n\n    I see, you're right.  Thanks, Gerrit.\n\n    ----- End forwarded message -----\n\n    This patch hard-codes the 3,4-argument cases in the way required by\n    POSIX.\n\n    Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>\n\nfunction                                             old     new   delta\ntest_main                                            370     421     +51\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/78b1b1b07ac2784f67bc65bb1d86cc2559a12446"},{"sha":"ec05df13b0f3bc69074909f078f981f417d95c89","author":{"name":"Denys Vlasenko","email":"8ed5e5289815171c75f140988d36b29c26205b2c@googlemail.com"},"message":"ash: align --login code with dash\n\nUpstream commit:\n\n    Date: Sun, 13 Jul 2008 22:34:50 +0800\n    [OPTIONS] Added support for -l\n\n    This patch adds support for the -l option (login shell) as required\n    by the LSB.\n\n    Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>\n\nIt's a bit bigger, but gets rid of one global variable\n\nfunction                                             old     new   delta\noptions                                              554     576     +22\n\nSigned-off-by: Denys Vlasenko <vda.linux@googlemail.com>","distinct":false,"url":"https://api.github.com/repos/rmyorston/busybox-w32/commits/ec05df13b0f3bc69074909f078f981f417d95c89"}]}
{"actor":{"display_login":"rmyorston"}}