xmake-repo icon indicating copy to clipboard operation
xmake-repo copied to clipboard

openmvg: add new package

Open choyy opened this issue 5 months ago • 4 comments

choyy avatar Jun 16 '25 02:06 choyy

Unbundle dependencies as much as possible

-- CEREAL: (internal)
-- EIGEN:  (external)
-- CERES: 1.13.0 (internal)
-- FLANN: 1.8.4 (internal)
-- LIBTIFF:  (internal)
-- LIBPNG:  (internal)
-- LIBJPEG (internal)
-- CLP: 1.16.10 (internal)
-- COINUTILS: 2.10.13 (internal)
-- OSI:  (internal)
-- LEMON: 1.3 (internal)

star-hengxing avatar Jun 24 '25 13:06 star-hengxing

We need to remove error pc file and let xmake automatically generate it (openmvg.pc)

$ ls /home/star/.xmake/packages/o/openmvg/2.1/d44a2d53c7564cc797ab9fd2028d7371/lib/pkgconfig
╭─────────┬────────────────────────────────────────────────────────────────────────────────────────────────────────┬────────────┬────────────┬────────────────────╮
│       # │                                                  name                                                  │    type    │    size    │      modified      │
├─────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────┼────────────┼────────────┼────────────────────┤
│       0 │ /home/star/.xmake/packages/o/openmvg/2.1/d44a2d53c7564cc797ab9fd2028d7371/lib/pkgconfig/flann.pc       │ file       │      392 B │ 2 minutes ago      │
╰─────────┴────────────────────────────────────────────────────────────────────────────────────────────────────────┴────────────┴────────────┴────────────────────╯

star-hengxing avatar Jun 24 '25 14:06 star-hengxing

We need to remove error pc file and let xmake automatically generate it (openmvg.pc)

$ ls /home/star/.xmake/packages/o/openmvg/2.1/d44a2d53c7564cc797ab9fd2028d7371/lib/pkgconfig
╭─────────┬────────────────────────────────────────────────────────────────────────────────────────────────────────┬────────────┬────────────┬────────────────────╮
│       # │                                                  name                                                  │    type    │    size    │      modified      │
├─────────┼────────────────────────────────────────────────────────────────────────────────────────────────────────┼────────────┼────────────┼────────────────────┤
│       0 │ /home/star/.xmake/packages/o/openmvg/2.1/d44a2d53c7564cc797ab9fd2028d7371/lib/pkgconfig/flann.pc       │ file       │      392 B │ 2 minutes ago      │
╰─────────┴────────────────────────────────────────────────────────────────────────────────────────────────────────┴────────────┴────────────┴────────────────────╯

这个咋做

choyy avatar Jun 25 '25 03:06 choyy

-- CEREAL: (internal)
-- EIGEN:  (external)
-- CERES: 1.13.0 (internal)
-- FLANN:  (external)
-- LIBTIFF: 4.7.0 (external)
-- LIBPNG: 1.6.48 (external)
-- LIBJPEG (external)
-- CLP: 1.16.10 (internal)
-- COINUTILS: 2.10.13 (internal)
-- OSI:  (internal)
-- LEMON:  (external)

TODO

  • cereal
  • ceres
  • clp
  • coinutils
  • osi

star-hengxing avatar Jun 26 '25 17:06 star-hengxing

Is it ready now?

waruqi avatar Jun 30 '25 10:06 waruqi

I don't know, it is probably awaiting 4 more dependencies to debundle and it would require to port coin-or-osi/clp for that case, as I ported coin-or-coinutils recently.

luadebug avatar Jun 30 '25 13:06 luadebug

How to fix openblas(macosx)?

choyy avatar Jul 12 '25 15:07 choyy

ld: warning: ignoring duplicate libraries: '-lpthread'
ld: warning: search path '/opt/homebrew/Cellar/gcc/14.2.0/lib/gcc/14/' not found
ld: library 'gfortran' not found
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [openblas_utest] Error 1
make: *** [tests] Error 2

I assume it would require to edit .YML file for MacOS X to solve that, as I do not know what exactly to edit and fix though?

luadebug avatar Jul 12 '25 15:07 luadebug

Cannot fix ceres(macosx, maybe also windows). Can you help with that? @luadebug @star-hengxing

choyy avatar Jul 14 '25 12:07 choyy

I don't have time to fix this problem. On Mac, you can choose to use its bundle dependency as workaround.

star-hengxing avatar Jul 14 '25 14:07 star-hengxing

Your CI paths are still not right. If it has something to do with macos CI, you can wait for me to have a look tomorrow. Parhaps, I know how to fix "library 'gfortran' not found".

Willaaaaaaa avatar Jul 14 '25 18:07 Willaaaaaaa

Your CI paths are still not right. If it has something to do with macos CI, you can wait for me to have a look tomorrow. Parhaps, I know how to fix "library 'gfortran' not found".

The problem "library 'gfortran' not found" has been fixed. Now the problem is with ceres package.

choyy avatar Jul 15 '25 00:07 choyy

How should we resolve current branch conflict @star-hengxing ? -s version is better ig?

luadebug avatar Jul 18 '25 15:07 luadebug

I think symbolic link is enough?

choyy avatar Jul 20 '25 13:07 choyy

It works now without setting symbols.

choyy avatar Jul 23 '25 11:07 choyy