<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 01/04/2012 11:19 AM, Vít Ondruch wrote:
    <blockquote cite="mid:4F047C04.8070709@redhat.com" type="cite">
      <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
      Dne 4.1.2012 16:31, Mo Morsi napsal(a):
      <blockquote cite="mid:4F0470E3.9050909@redhat.com" type="cite">
        <meta content="text/html; charset=UTF-8"
          http-equiv="Content-Type">
        On 01/02/2012 08:55 AM, Bohuslav Kabrda wrote:
        <blockquote
cite="mid:8909abdb-b305-4554-9cd7-a67ddc655c5b@zmail15.collab.prod.int.phx2.redhat.com"
          type="cite">
          <pre wrap="">Hi guys,
thank you all for your comments. I updated the guidelines draft to reflect them:</pre>
        </blockquote>
        <br>
        Again thanks for the new guidelines. Just a couple more comments
        inline below<br>
        <br>
        <blockquote
cite="mid:8909abdb-b305-4554-9cd7-a67ddc655c5b@zmail15.collab.prod.int.phx2.redhat.com"
          type="cite">
          <pre wrap="">- BR: ruby is now replaced with BR: ruby-devel for Ruby packages.</pre>
        </blockquote>
        <br>
        Possible duplication / discrepancy:<br>
        <br>
        - In 'Ruby Packaging Guidelines':<br>
        "Ruby packages <b>must</b> require ruby-devel package at build
        time with a <code>BuildRequires: ruby-devel</code>, and <b>may</b>
        indicate the minimal ruby version they need for building."<br>
        <br>
        - In 'Build Architecture and File Placement':<br>
        "All non-gem ruby packages <b>must</b> require ruby-devel
        package at build time with a <code>BuildRequires: ruby-devel</code>.
        "<br>
        <br>
        <br>
        Which should it be, 'all' ruby packages or just 'non-gem' ruby
        packages. Most likely the former, so for simplicity sake, the
        latter should be removed.<br>
      </blockquote>
      <br>
      The latter is correct, since gems requires rubygems-devel and that
      should be enough for gems.<br>
      <br>
    </blockquote>
    <br>
    Hrm? Don't see a rubygems-devel subpackage in the rubygems spec, is
    this going to be updated as well to incorporate that?<br>
    <br>
    <br>
    <br>
    <blockquote cite="mid:4F047C04.8070709@redhat.com" type="cite">
      However, reading the guidelines again, I am not happy with the
      structure. We have RubyGems section, but we don't have Non-gems
      section. The "Build Architecture and File Placement" should be
      probably 3rd level and we need some nice 2nd level caption. Any
      idea?<br>
    </blockquote>
    <br>
    Good idea, not sure as to the 2nd level though<br>
    <br>
      -Mo<br>
    <br>
    <br>
  </body>
</html>