#150 wxsqlite3.pc file needed

Subversion_TRUNK
closed-accepted
None
5
2012-12-26
2012-10-20
MartinKG
No

wxsqlite3 doesn't provide .pc file. It is good when ever -devel subpackage provides flags for gcc compiler.
https://fedoraproject.org/wiki/Packaging:Guidelines#Pkgconfig_Files_.28foo.pc.29

We are here to create a program package which compiles against wxSQLite3.

Could you please provide the attaches file in your next version ?

Discussion

  • MartinKG

    MartinKG - 2012-10-20

    wxsqlite3.pc

     
  • Ulrich Telle

    Ulrich Telle - 2012-10-20
    • assigned_to: nobody --> utelle
    • status: open --> open-accepted
     
  • Ulrich Telle

    Ulrich Telle - 2012-10-20

    Of course I can add this file to wxSQLite3 and distribute it with the next file release.

    Is it ok to put the pc file in the root folder of wxSQLite3? Or should it be located in a subfolder?

     
  • MartinKG

    MartinKG - 2012-10-20

    it's ok to put it into the root folder.

     
  • Ulrich Telle

    Ulrich Telle - 2012-11-24

    With the release of wxSQLite3 3.0.1 a .pc file is included.

    I might need advice if changes to this file should be required in the future.

     
  • Ulrich Telle

    Ulrich Telle - 2012-11-24
    • status: open-accepted --> closed-fixed
     
  • MartinKG

    MartinKG - 2012-12-23

    i reopened the case, because wxsqlite3.pc needs more corrections, please supply the file in a future version.

     
    • Ulrich Telle

      Ulrich Telle - 2012-12-23

      As I do most of my development on Windows platforms I have to admit that I don't know anything about .pc files.

      Of course I have no problem to replace the .pc file I added to the wxSQLite3 package, but I'm a bit confused by the changes introduced by the new version you supplied:

      Line "Requires: wxGTK-devel sqlite-devel" was dropped.
      wxSQLite3 depends on SQLite, so I would have expected that this prerequisite would remain in the .pc file.

      Line "Libs: -L${libdir} -lsqlite3" was replaced by
      "Libs: -lwxcode_gtk2u_wxsqlite3-2.8"
      Again the SQLite dependency was removed and replaced by a reference to wxSQLite3 itself - and obviously to a wxWidgets 2.8.x based one.

      wxSQLite3 supports wxWidgets 2.8.x and 2.9.x. Wouldn't the .pc file be invalid for someone using wxWidgets 2.9.x?

      Maybe you could write a PM to me clarifying what's going on here? Thanks.

      Regards,

      Ulrich

       
  • MartinKG

    MartinKG - 2012-12-26

    As we have already discussed outside, please add the attached
    pkgconfig skeleton file wxsqlite3-3.0.1.pc.in and remove the file wxsqlite3.pc. Subsequently, the necessary wxsqlite3 3.0.1.pc will be created on the part of Fedora.

     
  • MartinKG

    MartinKG - 2012-12-26

    added once again a corrected version of wxsqlite3-3.0.1.pc.in

     
  • Ulrich Telle

    Ulrich Telle - 2012-12-26

    I'll remove wxsqlite3.pc and add wxsqlite3.pc.in.

    I'd recommend to make not only the wxWidgets version a parameter from the outside, but also the build type (inserting "u" for Unicode and "d" for Debug into the library name as appropriate).

    I assume the latest version of wxsqlite3.pc.in fixes the linker issue reported in ticket #151, right?

     
    • MartinKG

      MartinKG - 2012-12-26

      that's true, the last version is ok.

       
  • Ulrich Telle

    Ulrich Telle - 2012-12-26
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,4 +1,3 @@
    -
     wxsqlite3 doesn't provide .pc file. It is good when ever -devel subpackage provides flags for gcc compiler.
     https://fedoraproject.org/wiki/Packaging:Guidelines\#Pkgconfig\_Files\_.28foo.pc.29
    
    • status: closed-fixed --> open-accepted
    • milestone: --> Subversion_TRUNK
     
  • Ulrich Telle

    Ulrich Telle - 2012-12-26

    The latest supplied version of wxsqlite3.pc.in was added to SVN (and the previous wxsqlite3.pc was removed).

     
  • Ulrich Telle

    Ulrich Telle - 2012-12-26
    • status: open-accepted --> closed-accepted
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks