Uploaded image for project: 'CUBRID Engine'
  1. CUBRID Engine
  2. ENGINE-15

Follow up with CUBRID submission to Fedora official repository

    Details

    • Type: Task
    • Status: Reopened
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Installer
    • Labels:
      None

      Description

      Hello Ovidiu,

      More than a year ago I have submitted a Review Request to Fedora community to add CUBRID to their official repository. See https://bugzilla.redhat.com/show_bug.cgi?id=658754.

      Since then we've made some clean-ups at SF.net that the links I indicated at that issue post are no longer valid.

      One of the developers seems to have looked at the request and yesterday replied that the links are broken.

      Will you please follow up this request and take care of Fedora submission?

      Thanks.

      Let me know if you have any questions.

      1. cubrid.spec
        17 kB
        Veliscu Ovidiu
      2. cubrid.spec
        17 kB
        Veliscu Ovidiu
      3. warnings.txt
        89 kB
        Veliscu Ovidiu
      4. warnings.txt
        17 kB
        Veliscu Ovidiu
      5. warnings.txt
        27 kB
        Veliscu Ovidiu
      6. warnings-demodb.txt
        0.8 kB
        Veliscu Ovidiu
      7. warnings-demodb.txt
        0.8 kB
        Veliscu Ovidiu
      8. warnings-devel.txt
        0.9 kB
        Veliscu Ovidiu
      9. warnings-devel.txt
        0.9 kB
        Veliscu Ovidiu
      10. warnings-spec.txt
        0.2 kB
        Veliscu Ovidiu
      11. warnings-spec.txt
        0.2 kB
        Veliscu Ovidiu

        Activity

        Hide
        kadishmal Esen Sagynov added a comment -

        Hi Veliscu Ovidiu, did you have a chance to go over the comments?

        Show
        kadishmal Esen Sagynov added a comment - Hi Veliscu Ovidiu , did you have a chance to go over the comments?
        Hide
        ovidiu.veliscu Veliscu Ovidiu added a comment -

        > ./autogen.sh
        > ./configure $

        {CUBRID_COMMON_CONFIGURE}

        autogen.sh but no %configure macro usage? See "rpm --eval %configure". If %configure cannot be used with this configure script, please add a comment in the spec file.

        Response: passing bindir parameter or libdir parameter will lead to an error (not supported feature).

        > %global debug_package %{nil}

        Why that?

        Response: I had to add a line "%global debug_package %{nil}", because it gave me a rpm building error "no build id note found in" for cub_cmhttpd.

        > %build
        > %ifarch x86_64
        > CFLAGS=" -m64 "
        > CUBRID_COMMON_CONFIGURE="${CUBRID_COMMON_CONFIGURE}

        --enable-64bit"
        > %endif

        What about other 64-bit targets?

        Response: Which are the other 64-bit targets you are referring to ?

        > %install

        It's odd that this section is so long. Why is it necessary to install/adjust so many things here after "make install"? Why doesn't "make install" do all that?

        Response: Currently, make install installs all files in one directory. There is no optimization for Fedora or Ubuntu or other operating systems. The install section copies the files to their correct directories (conf files in /etc/conf/cubrid, the variable files to /var/cubrid, the executables to /usr/bin, the libraries to /usr/lib or /usr/lib64 depending on architecture and so forth).

        > find %

        {buildroot}

        -size 0 -delete

        Hopefully you'll never need to include an empty file or touch one to include it as %ghost.

        Response: The source may have empty files (for example some optional configuration files). This was to remove warnings and possible empty files from the source.

        > The %files sections list many files with very generic file names (even headers such as %{_includedir}/dbi.h). It'll be a lot of fun to avoid conflicts with other packages.

        Response: What suggestion do you have for these file names (to avoid conflicts) ?

        I have updated the spec file here: http://sourceforge.net/projects/cubrid/files/CUBRID-9.1.0/Linux/Fedora-RPM/cubrid.spec/download

        I did not thoroughly test the changes though. The build seems to work.

        Show
        ovidiu.veliscu Veliscu Ovidiu added a comment - > ./autogen.sh > ./configure $ {CUBRID_COMMON_CONFIGURE} autogen.sh but no %configure macro usage? See "rpm --eval %configure". If %configure cannot be used with this configure script, please add a comment in the spec file. Response: passing bindir parameter or libdir parameter will lead to an error (not supported feature). > %global debug_package %{nil} Why that? Response: I had to add a line "%global debug_package %{nil}", because it gave me a rpm building error "no build id note found in" for cub_cmhttpd. > %build > %ifarch x86_64 > CFLAGS=" -m64 " > CUBRID_COMMON_CONFIGURE="${CUBRID_COMMON_CONFIGURE} --enable-64bit" > %endif What about other 64-bit targets? Response: Which are the other 64-bit targets you are referring to ? > %install It's odd that this section is so long. Why is it necessary to install/adjust so many things here after "make install"? Why doesn't "make install" do all that? Response: Currently, make install installs all files in one directory. There is no optimization for Fedora or Ubuntu or other operating systems. The install section copies the files to their correct directories (conf files in /etc/conf/cubrid, the variable files to /var/cubrid, the executables to /usr/bin, the libraries to /usr/lib or /usr/lib64 depending on architecture and so forth). > find % {buildroot} -size 0 -delete Hopefully you'll never need to include an empty file or touch one to include it as %ghost. Response: The source may have empty files (for example some optional configuration files). This was to remove warnings and possible empty files from the source. > The %files sections list many files with very generic file names (even headers such as %{_includedir}/dbi.h). It'll be a lot of fun to avoid conflicts with other packages. Response: What suggestion do you have for these file names (to avoid conflicts) ? I have updated the spec file here: http://sourceforge.net/projects/cubrid/files/CUBRID-9.1.0/Linux/Fedora-RPM/cubrid.spec/download I did not thoroughly test the changes though. The build seems to work.
        Hide
        kadishmal Esen Sagynov added a comment -

        Thank you for your time, Ovidiu. I've replied to Michael's comment at https://bugzilla.redhat.com/show_bug.cgi?id=658754#c39.

        I think we need to keep this spec file in a version control.

        Show
        kadishmal Esen Sagynov added a comment - Thank you for your time, Ovidiu. I've replied to Michael's comment at https://bugzilla.redhat.com/show_bug.cgi?id=658754#c39 . I think we need to keep this spec file in a version control.
        Hide
        kadishmal Esen Sagynov added a comment -

        Hello Veliscu Ovidiu,

        You may already know that we've found a Fedora sponsor. We've tried to build CUBRID on Fedora servers (Koji) and they are successful. However, Peter, the sponsor, has commented on several issues at https://bugzilla.redhat.com/show_bug.cgi?id=658754#c54. Would you please look at them? Thank you.

        Show
        kadishmal Esen Sagynov added a comment - Hello Veliscu Ovidiu , You may already know that we've found a Fedora sponsor. We've tried to build CUBRID on Fedora servers (Koji) and they are successful. However, Peter, the sponsor, has commented on several issues at https://bugzilla.redhat.com/show_bug.cgi?id=658754#c54 . Would you please look at them? Thank you.
        Hide
        kadishmal Esen Sagynov added a comment -

        Hi Veliscu Ovidiu,

        Did you have a chance to review the comment?

        Show
        kadishmal Esen Sagynov added a comment - Hi Veliscu Ovidiu , Did you have a chance to review the comment?

          People

          • Assignee:
            kadishmal Esen Sagynov
            Reporter:
            kadishmal Esen Sagynov
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: