First; Pull request #5 https://github.com/apache/apr/pull/5/files corresponds to https://bz.apache.org/bugzilla/show_bug.cgi?id=61985 Yann and I are neutral on dodging (fd == -1), what do others think? This would be a trivial one-line fix if we adopt it.
Second; Are we comfortable adding a dependency on autoconf-archive? This macro below dates back to 2010. If we are not, I can revert this. In any case, I think it would be unwise to introduce this before 1.7, we generally haven't changed our build prereqs for a subversion rebuild. On Wed, Aug 29, 2018 at 10:01 PM, <wr...@apache.org> wrote: > Author: wrowe > Date: Thu Aug 30 03:01:53 2018 > New Revision: 1839628 > > URL: http://svn.apache.org/viewvc?rev=1839628&view=rev > Log: > Submitted by: Hongxu Jia <hongxu....@windriver.com> > https://patch-diff.githubusercontent.com/raw/apache/apr/pull/8.patch > > While cross-compiling, the tools/gen_test_char could not > be executed at build time, use AX_PROG_CC_FOR_BUILD to > build native tools/gen_test_char > > Support explicit libtool by variable assigning before buildcheck.sh, > it is helpful for cross-compiling (such as libtool=aarch64-linux-libtool) > > Backports: r1839627 > > Modified: > apr/apr/branches/1.7.x/ (props changed) > apr/apr/branches/1.7.x/Makefile.in > apr/apr/branches/1.7.x/build/ (props changed) > apr/apr/branches/1.7.x/build/buildcheck.sh > apr/apr/branches/1.7.x/configure.in > > Propchange: apr/apr/branches/1.7.x/ > ------------------------------------------------------------ > ------------------ > --- svn:mergeinfo (original) > +++ svn:mergeinfo Thu Aug 30 03:01:53 2018 > @@ -1,4 +1,4 @@ > /apr/apr/branches/1.4.x:1003369,1101301 > -/apr/apr/trunk:733052,739635,741862,741866-741867,741869, > 741871,745763-745764,746310,747990,748080,748361,748371, > 748565,748888,748902,748988,749810,760443,767895,775683, > 782838,783398,783958,784633,784773,788588,789050,793192- > 793193,794118,794485,795267,799497,800627,809745,809854, > 810472,811455,813063,821306,829490,831641,832904,835607, > 888669,892028,892159,892435,892909,896382,896653,899905, > 901088,902077,902090,908427,910419,910597,917819,917837- > 917838,923311,923320,925965,929796,930508,931973,932585, > 951771,960665,960671,979891,983618,989450,990435,1003338, > 1044440,1044447,1055657,1072165,1078845,1081462,1081495,1083038,1083242, > 1084662,1086695,1088023,1089031,1089129,1089438,1099348,1103310,1183683, > 1183685-1183686,1183688,1183693,1183698,1213382,1235047,1236970,1237078, > 1237507,1240472,1340286,1340288,1340470,1341193,1341196,1343233,1343243, > 1367050,1368819,1370494,1372018,1372022,1372093,1372849,1376957,1384764, > 1389077,1400200,1402868,1405985,1406690,1420106, > 1420109,1425356,1428809,143 > 8940,1438957-1438959,1442903,1449568,1456418,1459994, > 1460179-1460180,1460241,1460399,1460405,1462738,1462813,1470186,1470348, > 1475509,1478905,1480067,1481262,1481265,1484271,1487796,1489517,1496407, > 1502804,1510354,1516261,1523384,1523479,1523484,1523505,1523521,1523604, > 1523613,1523615,1523844-1523845,1523853,1524014,1524031,1528797,1528809, > 1529488,1529495,1529515,1529521,1529668,1530786,1530800,1530988,1531554, > 1531768,1531884,1532022,1533104,1533111,1533979,1535027,1535157,1536744, > 1538171,1539374,1539389,1539455,1539603,1541054,1541061,1541486,1541655, > 1541666,1541744,1542601,1542779,1543033,1543056,1548575,1550907,1551650, > 1551659,1558905,1559382,1559873,1559975,1561040,1561260,1561265,1561321, > 1561347,1561356,1561361,1561394,1561555,1571894,1575509,1578420,1587045, > 1587063,1587543,1587545,1588878,1588937,1589982,1593611,1593614-1593615, > 1593680,1594684,1594708,1595549,1597797,1597803,1604590,1604596,1604598, > 1605104,1610854,1611023,1611107,1611110,1611117,1611120,1611125,1611184, > 1611193, > 1611466,1611515,1611517,1625173,1626564,1634615,1642159,1648830,1664406, > 1664447,1664451,1664471,1664769-1664770,1664775,1664904,1664911,1664958, > 1666341,1666411,1666458,1666611,1667420-1667421,1667423,1667900-1667901, > 1667903,1667914-1667916,1667962,1669077,1671292,1671329,1671356,1671386, > 1671389,1671513-1671514,1671957,1672354,1672366,1672495,1672575,1675644, > 1675656,1675668,1676013,1683521,1685929,1696140,1696767,1722547,1722557, > 1726928,1727020,1727160,1727175,1727199,1728957,1732582,1733451,1733594, > 1733694,1733706,1733708,1733775,1734816,1736552,1738791,1738925,1750374, > 1755709,1755740,1755746,1755758,1755954,1761279,1762326,1774712,1774973, > 1775069,1776994,1776998,1788334,1788337,1788929,1789947,1789998,1790045, > 1790200,1790296,1790302-1790304,1790330-1790331,1790436,1790439,1790444, > 1790446,1790488,1790521,1790523,1790569,1790632,1791598,1791718,1791728, > 1792621-1792622,1792625,1792961,1792963,1797415,1798105,1805380,1808039, > 1808836,1808910,1809649,1810452,1813286,1813330,1814239-18142 > 40,1814326,1814329,1814331,1816527,1816628,1817485, > 1819857-1819858,1819860-1819861,1819934-1819935,1820080,1820755,1822357, > 1827534,1832203,1832691,1832985,1834253,1834494,1834541,1836235,1839068, > 1839493,1839622 > +/apr/apr/trunk:733052,739635,741862,741866-741867,741869, > 741871,745763-745764,746310,747990,748080,748361,748371, > 748565,748888,748902,748988,749810,760443,767895,775683, > 782838,783398,783958,784633,784773,788588,789050,793192- > 793193,794118,794485,795267,799497,800627,809745,809854, > 810472,811455,813063,821306,829490,831641,832904,835607, > 888669,892028,892159,892435,892909,896382,896653,899905, > 901088,902077,902090,908427,910419,910597,917819,917837- > 917838,923311,923320,925965,929796,930508,931973,932585, > 951771,960665,960671,979891,983618,989450,990435,1003338, > 1044440,1044447,1055657,1072165,1078845,1081462,1081495,1083038,1083242, > 1084662,1086695,1088023,1089031,1089129,1089438,1099348,1103310,1183683, > 1183685-1183686,1183688,1183693,1183698,1213382,1235047,1236970,1237078, > 1237507,1240472,1340286,1340288,1340470,1341193,1341196,1343233,1343243, > 1367050,1368819,1370494,1372018,1372022,1372093,1372849,1376957,1384764, > 1389077,1400200,1402868,1405985,1406690,1420106, > 1420109,1425356,1428809,143 > 8940,1438957-1438959,1442903,1449568,1456418,1459994, > 1460179-1460180,1460241,1460399,1460405,1462738,1462813,1470186,1470348, > 1475509,1478905,1480067,1481262,1481265,1484271,1487796,1489517,1496407, > 1502804,1510354,1516261,1523384,1523479,1523484,1523505,1523521,1523604, > 1523613,1523615,1523844-1523845,1523853,1524014,1524031,1528797,1528809, > 1529488,1529495,1529515,1529521,1529668,1530786,1530800,1530988,1531554, > 1531768,1531884,1532022,1533104,1533111,1533979,1535027,1535157,1536744, > 1538171,1539374,1539389,1539455,1539603,1541054,1541061,1541486,1541655, > 1541666,1541744,1542601,1542779,1543033,1543056,1548575,1550907,1551650, > 1551659,1558905,1559382,1559873,1559975,1561040,1561260,1561265,1561321, > 1561347,1561356,1561361,1561394,1561555,1571894,1575509,1578420,1587045, > 1587063,1587543,1587545,1588878,1588937,1589982,1593611,1593614-1593615, > 1593680,1594684,1594708,1595549,1597797,1597803,1604590,1604596,1604598, > 1605104,1610854,1611023,1611107,1611110,1611117,1611120,1611125,1611184, > 1611193, > 1611466,1611515,1611517,1625173,1626564,1634615,1642159,1648830,1664406, > 1664447,1664451,1664471,1664769-1664770,1664775,1664904,1664911,1664958, > 1666341,1666411,1666458,1666611,1667420-1667421,1667423,1667900-1667901, > 1667903,1667914-1667916,1667962,1669077,1671292,1671329,1671356,1671386, > 1671389,1671513-1671514,1671957,1672354,1672366,1672495,1672575,1675644, > 1675656,1675668,1676013,1683521,1685929,1696140,1696767,1722547,1722557, > 1726928,1727020,1727160,1727175,1727199,1728957,1732582,1733451,1733594, > 1733694,1733706,1733708,1733775,1734816,1736552,1738791,1738925,1750374, > 1755709,1755740,1755746,1755758,1755954,1761279,1762326,1774712,1774973, > 1775069,1776994,1776998,1788334,1788337,1788929,1789947,1789998,1790045, > 1790200,1790296,1790302-1790304,1790330-1790331,1790436,1790439,1790444, > 1790446,1790488,1790521,1790523,1790569,1790632,1791598,1791718,1791728, > 1792621-1792622,1792625,1792961,1792963,1797415,1798105,1805380,1808039, > 1808836,1808910,1809649,1810452,1813286,1813330,1814239-18142 > 40,1814326,1814329,1814331,1816527,1816628,1817485, > 1819857-1819858,1819860-1819861,1819934-1819935,1820080,1820755,1822357, > 1827534,1832203,1832691,1832985,1834253,1834494,1834541,1836235,1839068, > 1839493,1839622,1839627 > /apr/apr/trunk/test/testnames.c:1460405 > /httpd/httpd/trunk:1604590 > > Modified: apr/apr/branches/1.7.x/Makefile.in > URL: http://svn.apache.org/viewvc/apr/apr/branches/1.7.x/ > Makefile.in?rev=1839628&r1=1839627&r2=1839628&view=diff > ============================================================ > ================== > --- apr/apr/branches/1.7.x/Makefile.in (original) > +++ apr/apr/branches/1.7.x/Makefile.in Thu Aug 30 03:01:53 2018 > @@ -46,7 +46,7 @@ LT_VERSION = @LT_VERSION@ > > CLEAN_TARGETS = apr-config.out apr.exp exports.c export_vars.c .make.dirs > \ > build/apr_rules.out tools/gen_test_char@EXEEXT@ \ > - tools/gen_test_char.o tools/gen_test_char.lo \ > + tools/gen_test_char.o \ > include/private/apr_escape_test_char.h > DISTCLEAN_TARGETS = config.cache config.log config.status \ > include/apr.h include/arch/unix/apr_private.h \ > @@ -130,13 +130,9 @@ check: $(TARGET_LIB) > etags: > etags `find . -name '*.[ch]'` > > -OBJECTS_gen_test_char = tools/gen_test_char.lo $(LOCAL_LIBS) > -tools/gen_test_char.lo: tools/gen_test_char.c > +tools/gen_test_char@EXEEXT@: tools/gen_test_char.c > $(APR_MKDIR) tools > - $(LT_COMPILE) > - > -tools/gen_test_char@EXEEXT@: $(OBJECTS_gen_test_char) > - $(LINK_PROG) $(OBJECTS_gen_test_char) $(ALL_LIBS) > + $(CC_FOR_BUILD) $(CFLAGS_FOR_BUILD) $< -o $@ > > include/private/apr_escape_test_char.h: tools/gen_test_char@EXEEXT@ > $(APR_MKDIR) include/private > > Propchange: apr/apr/branches/1.7.x/build/ > ------------------------------------------------------------ > ------------------ > --- svn:mergeinfo (original) > +++ svn:mergeinfo Thu Aug 30 03:01:53 2018 > @@ -1,4 +1,4 @@ > /apr/apr/branches/1.4.x/build:1003369,1101301 > -/apr/apr/trunk/build:733052,739635,741862,741866-741867, > 741869,741871,745763-745764,746310,747990,748080,748361, > 748371,748565,748888,748902,748988,749810,760443,767895, > 775683,782838,783398,783958,784633,784773,788588,789050, > 793192-793193,794118,794485,795267,799497,800627,809745, > 809854,810472,811455,813063,821306,829490,831641,832904, > 835607,888669,892028,892159,892435,892909,896382,896653, > 899905,901088,902077,902090,908427,910419,910597,917819, > 917837-917838,923311,923320,925965,929796,930508,931973, > 932585,951771,960665,960671,979891,983618,989450,990435, > 1003338,1044440,1044447,1055657,1072165,1078845,1081462,1081495,1083038, > 1083242,1084662,1086695,1088023,1089031,1089129,1089438,1099348,1103310, > 1183683,1183685-1183686,1183688,1183693,1183698,1213382,1235047,1236970, > 1237078,1237507,1240472,1340286,1340288,1340470,1341193,1341196,1343233, > 1343243,1367050,1368819,1370494,1372018,1372022,1372093,1372849,1376957, > 1384764,1389077,1400200,1402868,1405985,1406690, > 1420106,1420109,1425356,14288 > 09,1438940,1438957-1438959,1442903,1449568,1456418, > 1459994,1460179-1460180,1460241,1460399,1460405,1462738,1462813,1470186, > 1470348,1475509,1478905,1480067,1481262,1481265,1484271,1487796,1489517, > 1496407,1502804,1510354,1516261,1523384,1523479,1523484,1523505,1523521, > 1523604,1523613,1523615,1523844-1523845,1523853,1524014,1524031,1528797, > 1528809,1529488,1529495,1529515,1529521,1529668,1530786,1530800,1530988, > 1531554,1531768,1531884,1532022,1533104,1533111,1533979,1535027,1535157, > 1536744,1538171,1539374,1539389,1539455,1539603,1541054,1541061,1541486, > 1541655,1541666,1541744,1542601,1542779,1543033,1543056,1548575,1550907, > 1551650,1551659,1558905,1559382,1559873,1559975,1561040,1561260,1561265, > 1561321,1561347,1561356,1561361,1561394,1561555,1571894,1575509,1578420, > 1587045,1587063,1587543,1587545,1588878,1588937,1589982,1593611,1593614- > 1593615,1593680,1594684,1594708,1595549,1597797,1597803,1604590,1604596, > 1604598,1605104,1610854,1611023,1611107,1611110,1611117,1611120,1611125, > 1611184,16 > 11193,1611466,1611515,1611517,1625173,1626564,1634615, > 1642159,1648830,1664406,1664447,1664451,1664471,1664769-1664770,1664775, > 1664904,1664911,1664958,1666341,1666411,1666458,1666611,1667420-1667421, > 1667423,1667900-1667901,1667903,1667914-1667916,1667962,1669077,1671292, > 1671329,1671356,1671386,1671389,1671513-1671514,1671957,1672354,1672366, > 1672495,1672575,1675644,1675656,1675668,1676013,1683521,1685929,1696140, > 1696767,1722547,1722557,1726928,1727020,1727160,1727175,1727199,1728957, > 1732582,1733451,1733594,1733694,1733706,1733708,1733775,1734816,1736552, > 1738791,1738925,1750374,1755709,1755740,1755746,1755758,1755954,1761279, > 1762326,1774712,1774973,1775069,1776994,1776998,1788334,1788337,1788929, > 1789947,1789998,1790045,1790200,1790296,1790302-1790304,1790330-1790331, > 1790436,1790439,1790444,1790446,1790488,1790521,1790523,1790569,1790632, > 1791598,1791718,1791728,1792621-1792622,1792625,1792961,1792963,1797415, > 1798105,1805380,1808039,1808836,1808910,1809649,1810452,1813286,1813330, > 1816527 > ,1816628,1817485,1819857-1819858,1819860-1819861,1819934-1819935,1820080, > 1820186,1834494 > +/apr/apr/trunk/build:733052,739635,741862,741866-741867, > 741869,741871,745763-745764,746310,747990,748080,748361, > 748371,748565,748888,748902,748988,749810,760443,767895, > 775683,782838,783398,783958,784633,784773,788588,789050, > 793192-793193,794118,794485,795267,799497,800627,809745, > 809854,810472,811455,813063,821306,829490,831641,832904, > 835607,888669,892028,892159,892435,892909,896382,896653, > 899905,901088,902077,902090,908427,910419,910597,917819, > 917837-917838,923311,923320,925965,929796,930508,931973, > 932585,951771,960665,960671,979891,983618,989450,990435, > 1003338,1044440,1044447,1055657,1072165,1078845,1081462,1081495,1083038, > 1083242,1084662,1086695,1088023,1089031,1089129,1089438,1099348,1103310, > 1183683,1183685-1183686,1183688,1183693,1183698,1213382,1235047,1236970, > 1237078,1237507,1240472,1340286,1340288,1340470,1341193,1341196,1343233, > 1343243,1367050,1368819,1370494,1372018,1372022,1372093,1372849,1376957, > 1384764,1389077,1400200,1402868,1405985,1406690, > 1420106,1420109,1425356,14288 > 09,1438940,1438957-1438959,1442903,1449568,1456418, > 1459994,1460179-1460180,1460241,1460399,1460405,1462738,1462813,1470186, > 1470348,1475509,1478905,1480067,1481262,1481265,1484271,1487796,1489517, > 1496407,1502804,1510354,1516261,1523384,1523479,1523484,1523505,1523521, > 1523604,1523613,1523615,1523844-1523845,1523853,1524014,1524031,1528797, > 1528809,1529488,1529495,1529515,1529521,1529668,1530786,1530800,1530988, > 1531554,1531768,1531884,1532022,1533104,1533111,1533979,1535027,1535157, > 1536744,1538171,1539374,1539389,1539455,1539603,1541054,1541061,1541486, > 1541655,1541666,1541744,1542601,1542779,1543033,1543056,1548575,1550907, > 1551650,1551659,1558905,1559382,1559873,1559975,1561040,1561260,1561265, > 1561321,1561347,1561356,1561361,1561394,1561555,1571894,1575509,1578420, > 1587045,1587063,1587543,1587545,1588878,1588937,1589982,1593611,1593614- > 1593615,1593680,1594684,1594708,1595549,1597797,1597803,1604590,1604596, > 1604598,1605104,1610854,1611023,1611107,1611110,1611117,1611120,1611125, > 1611184,16 > 11193,1611466,1611515,1611517,1625173,1626564,1634615, > 1642159,1648830,1664406,1664447,1664451,1664471,1664769-1664770,1664775, > 1664904,1664911,1664958,1666341,1666411,1666458,1666611,1667420-1667421, > 1667423,1667900-1667901,1667903,1667914-1667916,1667962,1669077,1671292, > 1671329,1671356,1671386,1671389,1671513-1671514,1671957,1672354,1672366, > 1672495,1672575,1675644,1675656,1675668,1676013,1683521,1685929,1696140, > 1696767,1722547,1722557,1726928,1727020,1727160,1727175,1727199,1728957, > 1732582,1733451,1733594,1733694,1733706,1733708,1733775,1734816,1736552, > 1738791,1738925,1750374,1755709,1755740,1755746,1755758,1755954,1761279, > 1762326,1774712,1774973,1775069,1776994,1776998,1788334,1788337,1788929, > 1789947,1789998,1790045,1790200,1790296,1790302-1790304,1790330-1790331, > 1790436,1790439,1790444,1790446,1790488,1790521,1790523,1790569,1790632, > 1791598,1791718,1791728,1792621-1792622,1792625,1792961,1792963,1797415, > 1798105,1805380,1808039,1808836,1808910,1809649,1810452,1813286,1813330, > 1816527 > ,1816628,1817485,1819857-1819858,1819860-1819861,1819934-1819935,1820080, > 1820186,1834494,1839627 > /apr/apr/trunk/test/testnames.c/build:1460405 > /httpd/httpd/trunk/build:1604590 > > Modified: apr/apr/branches/1.7.x/build/buildcheck.sh > URL: http://svn.apache.org/viewvc/apr/apr/branches/1.7.x/build/ > buildcheck.sh?rev=1839628&r1=1839627&r2=1839628&view=diff > ============================================================ > ================== > --- apr/apr/branches/1.7.x/build/buildcheck.sh (original) > +++ apr/apr/branches/1.7.x/build/buildcheck.sh Thu Aug 30 03:01:53 2018 > @@ -40,7 +40,9 @@ fi > # output is multiline from 1.5 onwards > > # Require libtool 1.4 or newer > -libtool=`build/PrintPath glibtool1 glibtool libtool libtool15 libtool14` > +if test -z "$libtool"; then > + libtool=`build/PrintPath glibtool1 glibtool libtool libtool15 libtool14` > +fi > lt_pversion=`$libtool --version 2>/dev/null|sed -e > 's/([^)]*)//g;s/^[^0-9]*//;s/[- ].*//g;q'` > if test -z "$lt_pversion"; then > echo "buildconf: libtool not found." > > Modified: apr/apr/branches/1.7.x/configure.in > URL: http://svn.apache.org/viewvc/apr/apr/branches/1.7.x/ > configure.in?rev=1839628&r1=1839627&r2=1839628&view=diff > ============================================================ > ================== > --- apr/apr/branches/1.7.x/configure.in (original) > +++ apr/apr/branches/1.7.x/configure.in Thu Aug 30 03:01:53 2018 > @@ -183,6 +183,9 @@ dnl can only be used once within a confi > dnl preload section from invoking the macro to get compiler info. > AC_PROG_CC > > +dnl Check build CC for gen_test_char compiling which is executed at build > time. > +AX_PROG_CC_FOR_BUILD > + > dnl AC_PROG_SED is only avaliable in recent autoconf versions. > dnl Use AC_CHECK_PROG instead if AC_PROG_SED is not present. > ifdef([AC_PROG_SED], > > >