<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Waiting for a build so I thought I'd
      comment on this commit.<br>
      <br>
      My comments are about style and maintenance not functionality <br>
      which others have covered (the gentoo comment).<br>
      <br>
      On 2017-10-12 02:35 PM, <a class="moz-txt-link-abbreviated" href="mailto:leonardo.sandoval.gonzalez@linux.intel.com">leonardo.sandoval.gonzalez@linux.intel.com</a>
      wrote:<br>
    </div>
    <blockquote type="cite"
cite="mid:20171012183523.130193-2-leonardo.sandoval.gonzalez@linux.intel.com">
      <pre wrap="">From: Leonardo Sandoval <a class="moz-txt-link-rfc2396E" href="mailto:leonardo.sandoval.gonzalez@linux.intel.com" moz-do-not-send="true">&lt;leonardo.sandoval.gonzalez@linux.intel.com&gt;</a>

Highlights:

    * Patches rebased to 5.26.1 and just one removed</pre>
    </blockquote>
    That leaves 5 patches that are 'pending'. <br>
    <br>
    Robert, Hongxu, Wenzong,<br>
    You guys added these upstream pending patches.<br>
    Are the patches you send still pending or were they rejected?<br>
    <br>
    Please elect one of the three of you to follow-up with a patch<br>
    if the status has changed and otherwise ping upstream perl<br>
    to get these patches merged so we can drop them locally.<br>
    <br>
    <blockquote type="cite"
cite="mid:20171012183523.130193-2-leonardo.sandoval.gonzalez@linux.intel.com">
      <pre wrap="">

    * Several configs values taken from upstream script uconfig[64].sh were introduced in
    config.sh, avoiding pre-processor issues.

    * Several libraries paths now included a './' as prefix, because @INC does not
    include local paths.

    * Manifypods is excluded from compilation due to compilation errors (see patch for
    more info)

    * New perl creates a static library (&lt;libdir&gt;/auto/Unicode/Normalize/Normalize.a) thus
    there is a new staticdev package.

New packages included (compared to previous version):

    perl-module-math-bigint-lib
    perl-module-test2
    perl-module-test2-api
    perl-module-test2-api-breakage
    perl-module-test2-api-context
    perl-module-test2-api-instance
    perl-module-test2-api-stack
    perl-module-test2-event
    perl-module-test2-event-bail
    perl-module-test2-event-diag
    perl-module-test2-event-encoding
    perl-module-test2-event-exception
    perl-module-test2-event-generic
    perl-module-test2-event-info
    perl-module-test2-event-note
    perl-module-test2-event-ok
    perl-module-test2-event-plan
    perl-module-test2-event-skip
    perl-module-test2-event-subtest
    perl-module-test2-event-tap-version
    perl-module-test2-event-waiting
    perl-module-test2-formatter
    perl-module-test2-formatter-tap
    perl-module-test2-hub
    perl-module-test2-hub-interceptor
    perl-module-test2-hub-interceptor-terminator
    perl-module-test2-hub-subtest
    perl-module-test2-ipc
    perl-module-test2-ipc-driver
    perl-module-test2-ipc-driver-files
    perl-module-test2-tools-tiny
    perl-module-test2-util
    perl-module-test2-util-externalmeta
    perl-module-test2-util-hashbase
    perl-module-test2-util-trace</pre>
    </blockquote>
    <br>
    The list of these test2 modules could be shorted to <br>
    <br>
    <pre wrap="">    perl-module-test2
    perl-module-test2-api
    ...
    perl-module-test2-util-trace

but that's probably long long bikeshedding
so at least your shed is painted.

</pre>
    <blockquote type="cite"
cite="mid:20171012183523.130193-2-leonardo.sandoval.gonzalez@linux.intel.com">
      <pre wrap="">
    perl-module-test-builder-formatter
    perl-module-test-builder-tododiag
    perl-module-unicode-collate-locale-de-at-ph
    perl-module-unicode-collate-locale-fr-ca
    perl-module-unicode-collate-locale-he
    perl-module-unicode-collate-locale-ug-cyrl
    perl-module-unicode-collate-locale-vo
    perl-module-unicode-normalize-staticdev

Signed-off-by: Leonardo Sandoval <a class="moz-txt-link-rfc2396E" href="mailto:leonardo.sandoval.gonzalez@linux.intel.com" moz-do-not-send="true">&lt;leonardo.sandoval.gonzalez@linux.intel.com&gt;</a></pre>
    </blockquote>
    <p><br>
    </p>
    <p>I like that your package update log has a summary of what's
      changed *in* the package,</p>
    <p>in addition to what's changed in the recipe. It would be nice if
      more people did that.</p>
    <p><br>
    </p>
    <p>Shame about the perl madness as reported by the gentoo devs.</p>
    <pre class="moz-signature" cols="72">-- 
# Randy MacLeod.  WR Linux
# Wind River an Intel Company
</pre>
  </body>
</html>