Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Some debian package manager tweaks#16733

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
alalek merged 3 commits intoopencv:masterfromrajpratik71:master
Mar 18, 2020
Merged

Conversation

@rajpratik71
Copy link
Contributor

By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages .

By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.

This results in smaller downloads and installation of packages .

Refer to blog atUbuntu Blog .

results in smaller image size.

Pull Request Readiness Checklist

See details athttps://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under OpenCV (BSD) License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages . By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.This results in smaller downloads and installation of packages .Refer to blog at [Ubuntu Blog](https://ubuntu.com/blog/we-reduced-our-docker-images-by-60-with-no-install-recommends) .results in smaller image size.
By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages . By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.This results in smaller downloads and installation of packages .Refer to blog at [Ubuntu Blog](https://ubuntu.com/blog/we-reduced-our-docker-images-by-60-with-no-install-recommends) .results in smaller image size.
By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages . By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.This results in smaller downloads and installation of packages .Refer to blog at [Ubuntu Blog](https://ubuntu.com/blog/we-reduced-our-docker-images-by-60-with-no-install-recommends) .results in smaller image size.
@alalek
Copy link
Member

Did you validate that scripts which use this containers are not broken by this patch?

@rajpratik71
Copy link
ContributorAuthor

Did you validate that scripts which use this containers are not broken by this patch?

build is still pending , i think needed some approval .

From my side , this is apt-get package install optimization , this tweak doesn't let install unnecessary packages or recommended packages in simple straight forward way

@alalek
Copy link
Member

install unnecessary packages or recommended packages

But these packages can be used by dependent scripts, so patch should be validated.

Unfortunately, there are no automatic tests on this stuff (it just a sample, no frequent updates are expected). So it requires manual validation on local machines.

@rajpratik71
Copy link
ContributorAuthor

By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages .

By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.

This results in smaller downloads and installation of packages .

Refer to blog atUbuntu Blog .

results in smaller image size.

Pull Request Readiness Checklist

See details athttps://github.com/opencv/opencv/wiki/How_to_contribute#making-a-good-pull-request

  • I agree to contribute to the project under OpenCV (BSD) License.
  • To the best of my knowledge, the proposed patch is not based on a code under GPL or other license that is incompatible with OpenCV
  • The PR is proposed to proper branch
  • There is reference to original bug report and related work
  • There is accuracy test, performance test and test data in opencv_extra repository, if applicable
    Patch to opencv_extra has the same branch name.
  • The feature is well documented and sample code can be built with the project CMake

As i mentioned and refered to the blog also . In general any apt package installation consist there a list of their main dependencies beside that there are some suggested packages , which "--no-install-recommends" prevent them from installing

@asmorkalov
Copy link
Contributor

@alalek It looks like the change does not introduce regressions.
Command:./build_plugins.sh ./tmp/
Output:

libopencv_videoio_ffmpeg_shared_2.8.15.solibopencv_videoio_ffmpeg_shared_4.1.so libopencv_videoio_ffmpeg_static_3.4.5.solibopencv_videoio_ffmpeg_ubuntu_16.04.solibopencv_videoio_gstreamer.solibopencv_videoio_ffmpeg_shared_3.4.5.solibopencv_videoio_ffmpeg_static_2.8.15.solibopencv_videoio_ffmpeg_static_4.1.solibopencv_videoio_ffmpeg_ubuntu_18.04.so
alalek reacted with thumbs up emoji

@asmorkalov
Copy link
Contributor

👍

@alalekalalek merged commit2160f9b intoopencv:masterMar 18, 2020
a-sajjad72 pushed a commit to a-sajjad72/opencv that referenced this pull requestMar 30, 2023
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@asmorkalovasmorkalovasmorkalov approved these changes

Assignees

@asmorkalovasmorkalov

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rajpratik71@alalek@asmorkalov

[8]ページ先頭

©2009-2025 Movatter.jp