Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

adjust depends according to namcap. #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sukanka
Copy link

@sukanka sukanka commented Apr 25, 2022

for output of namcap, see https://fars.ee/53NB

Some of the depdends should go to optdepends, but namcap cannot tell

I will further build this package using extra-x86_64-build or multilib-build or archlinuxcn-x86_64-build
to check the dependencies.

@Rubo3
Copy link
Owner

Rubo3 commented Apr 25, 2022

Thank you, this is a much needed PR. If you don't mind I'll wait before merging your changes, so that we take the time needed to test them and sort out the possible issues.

@sukanka
Copy link
Author

sukanka commented Apr 25, 2022

If you don't mind I'll wait before merging your changes, so that we take the time needed to test them and sort out the possible issues.

Sure, we should test them first.

I just ran archlinuxcn-x86_64-build, everything goes fine, but we need pay attention to the following output.

==> Checking for packaging issues...
==> WARNING: Package contains reference to $srcdir
usr/lib/matlab/bin/glnxa64/libmwsl_interface_editor_metamodel.so
usr/lib/matlab/bin/glnxa64/libmwsl_sysarch_metamodel_mi.so
usr/lib/matlab/bin/glnxa64/libmwsl_sysarch_metamodel.so

Maybe we should use sed to fix this warning?

@sukanka
Copy link
Author

sukanka commented Apr 26, 2022

I build it using extra-x86_64-build, then install the package and run namcap to check. Full output can be found at https://fars.ee/ti4i

BTW, extra-x86_64-build should also perform namcap, but I got the following error ( I have about 500GiB left, maybe I'm running out of memory (64GiB) or /tmp (32GiB) )

zstd: error 70 : Write error : cannot write decoded block : No space left on device 
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/usr/lib/python3.10/site-packages/namcap.py", line 245, in <module>
    process_realpackage(package, active_modules)
  File "/usr/lib/python3.10/site-packages/namcap.py", line 82, in process_realpackage
    pkgtar = open_package(package)
  File "/usr/lib/python3.10/site-packages/namcap.py", line 55, in open_package
    if '.PKGINFO' not in tar.getnames():
  File "/usr/lib/python3.10/tarfile.py", line 1820, in getnames
    return [tarinfo.name for tarinfo in self.getmembers()]
  File "/usr/lib/python3.10/tarfile.py", line 1812, in getmembers
    self._load()        # all members, we first have to
  File "/usr/lib/python3.10/tarfile.py", line 2410, in _load
    tarinfo = self.next()
  File "/usr/lib/python3.10/tarfile.py", line 2334, in next
    raise ReadError("unexpected end of data")
tarfile.ReadError: unexpected end of data

@Rubo3
Copy link
Owner

Rubo3 commented Apr 26, 2022

we need pay attention to the following output

Do you get the same output running just makepkg?

maybe I'm running out of memory

Strange, my directory weighs around 16 GiB and I don't have this problem.
Anyway, thank you again, I'll build it using the same tools you used and I'll let you know the results.

@sukanka
Copy link
Author

sukanka commented Apr 27, 2022

we need pay attention to the following output

Do you get the same output running just makepkg?

maybe I'm running out of memory

Strange, my directory weighs around 16 GiB and I don't have this problem. Anyway, thank you again, I'll build it using the same tools you used and I'll let you know the results.

No,I now know the problem, matlab is not in the extra repository. So extra-build doesn't fully work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants