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.

Sign up for GitHub

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

Jump to bottom

readline no longer a DEPENDS for jack 1.19.22 on the master branch #732

Closed
large2021 opened this issue Aug 15, 2023 · 7 comments
Closed

readline no longer a DEPENDS for jack 1.19.22 on the master branch #732

large2021 opened this issue Aug 15, 2023 · 7 comments

Comments

@large2021
Copy link
Contributor

For recipe meta-oe/recipes-multimedia/jack/jack_1.19.22.bb, on the "master" branch, the DEPENDS includes "readline". Since "jack-utils" is no longer being built, this dependency is obsolete.

As "readline" is GPLv3, this introduces the need to "DEPENDS:remove" the dependency for those of us who strive to be GPLv3-free.
@large2021
Copy link
Contributor Author

Also present on the "mickledore" branch.

@kraj
Copy link
Contributor

kraj commented Aug 15, 2023

@large2021 thanks for the report. Are you also working on a patch to fix this. ?

@large2021
Copy link
Contributor Author

large2021 commented Aug 15, 2023 via email

@kraj
Copy link
Contributor

kraj commented Aug 15, 2023

I’d be happy to submit a patch, and am capable of doing so, but am unfamiliar with the openembedded process/requirements. Please advise. Kindest, Kirk Hays From: Khem Raj @.> Sent: Tuesday, August 15, 2023 9:25 AM To: openembedded/meta-openembedded @.> Cc: Kirk Hays @.>; Mention @.> Subject: Re: [openembedded/meta-openembedded] readline no longer a DEPENDS for jack 1.19.22 on the master branch (Issue #732) @large2021<https://urldefense.com/v3/https:/github.com/large2021;!!HBnMciuwfVSXJQ!RON4-XEJHkHvzQhnOSD7DjvOta1CkGCZb73Mdcd-N6-hn3RuPY-14bmtUxxUzzAIPzAjXImM8HZcGjtpXlvoLgJ-$> thanks for the report. Are you also working on a patch to fix this. ? — Reply to this email directly, view it on GitHub<https://urldefense.com/v3/https:/github.com/openembedded/meta-openembedded/issues/732*issuecomment-1679251106;Iw!!HBnMciuwfVSXJQ!RON4-XEJHkHvzQhnOSD7DjvOta1CkGCZb73Mdcd-N6-hn3RuPY-14bmtUxxUzzAIPzAjXImM8HZcGjtpXnrBkY11$>, or unsubscribe<https://urldefense.com/v3/https:/github.com/notifications/unsubscribe-auth/AS7HQGB534VSMAIUSFKRHNLXVOPE5ANCNFSM6AAAAAA3RHPQOI;!!HBnMciuwfVSXJQ!RON4-XEJHkHvzQhnOSD7DjvOta1CkGCZb73Mdcd-N6-hn3RuPY-14bmtUxxUzzAIPzAjXImM8HZcGjtpXliRzw-H$>. You are receiving this because you were mentioned.Message ID: @.@.>>

you can create a patch and send to [email protected], you can also create a github pull which I can take to the mailing list.

@large2021
Copy link
Contributor Author

A patch has been submitted to the mailing list.

@kraj
Copy link
Contributor

kraj commented Aug 15, 2023

mailing lists are subscriber only, non-subscriber posts needs to be channelled by a moderator, so it may take a while for your post to reach the mailing list distribution.

@kraj
Copy link
Contributor

kraj commented Aug 18, 2023

fixed with 4bacc11

@kraj kraj closed this as completed Aug 18, 2023
PheeL79 pushed a commit to PheeL79/meta-openembedded that referenced this issue Aug 29, 2023
@wangmingyu84 @kraj
bats: upgrade 1.9.0 -> 1.10.0
1972581 
Changelog:
===========
Added:
-----------
 add ${BATS_TEST_TAGS[@]} for querying the tags during a test (openembedded#705)
 print tags on failing tests (openembedded#705)
 test for negative arguments to --jobs (openembedded#693)
 add tests for --formatter cat (openembedded#710)
 test coverage in CI (openembedded#718)
 Support for rush as alternative to GNU parallel (openembedded#729)
 add bats_pipe helper function for run that executes \| as pipes (openembedded#663)
 publish docker images to ghcr.io (additionally to Dockerhub) (openembedded#740)

Fixed:
---------
 fix run with options overwriting the value of i (openembedded#726, openembedded#727)
 fix ${BATS_TEST_NAMES[@]} containing only --tags instead of test name since Bats v1.8.0 (openembedded#705)
 fix run --keep-empty-lines counting trailing \n as (empty) new line (openembedded#711)
 fix short flag unpacker creating bogus command lines with valued flags (openembedded#732)
 fix formatter becoming confused with retries (openembedded#734)
 fix --gather-test-outputs-in fails on tests with / (openembedded#735)
 fix overriding date breaks --timing (openembedded#736)

Signed-off-by: Wang Mingyu <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Nov 19, 2024
@kraj
c-ares: upgrade 1.27.0 -> 1.28.1
fec9861 
Changelog 1.28.1 :

  This release contains a fix for a single significant regression introduced
  in c-ares 1.28.0.

  - ares_search() and ares_getaddrinfo() resolution fails if no search domains
    are specified. Issue openembedded#737

Changelog 1.28.0 :

  Features:

  - Emit warnings when deprecated c-ares functions are used. This can be
    disabled by passing a compiler definition of CARES_NO_DEPRECATED. PR openembedded#732
  - Add function ares_search_dnsrec() to search for records using the new DNS
    record data structures. PR openembedded#719
  - Rework internals to pass around ares_dns_record_t instead of binary data,
    this introduces new public functions of ares_query_dnsrec() and
    ares_send_dnsrec(). PR openembedded#730

Changes:

  - tests: when performing simulated queries, reduce timeouts to make tests run
    faster
  - Replace configuration file parsers with memory-safe parser. PR openembedded#725
  - Remove acountry completely, the manpage might still get installed otherwise. Issue openembedded#718

Bugfixes:

  - CMake: don't overwrite global required libraries/definitions/includes which
    could cause build errors for projects chain building c-ares. Issue openembedded#729
  - On some platforms, netinet6/in6.h is not included by netinet/in.h
    and needs to be included separately. PR openembedded#728
  - Fix a potential memory leak in ares_init(). Issue openembedded#724
  - Some platforms don't have the isascii() function. Implement as a macro. PR openembedded#721
  - CMake: Fix Chain building if CMAKE runtime paths not set
  - NDots configuration should allow a value of zero. PR openembedded#735

Signed-off-by: Jason Schonberg <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Nov 20, 2024
@kraj
c-ares: upgrade 1.27.0 -> 1.28.1
c8a35cf 
Changelog 1.28.1 :

  This release contains a fix for a single significant regression introduced
  in c-ares 1.28.0.

  - ares_search() and ares_getaddrinfo() resolution fails if no search domains
    are specified. Issue openembedded#737

Changelog 1.28.0 :

  Features:

  - Emit warnings when deprecated c-ares functions are used. This can be
    disabled by passing a compiler definition of CARES_NO_DEPRECATED. PR openembedded#732
  - Add function ares_search_dnsrec() to search for records using the new DNS
    record data structures. PR openembedded#719
  - Rework internals to pass around ares_dns_record_t instead of binary data,
    this introduces new public functions of ares_query_dnsrec() and
    ares_send_dnsrec(). PR openembedded#730

Changes:

  - tests: when performing simulated queries, reduce timeouts to make tests run
    faster
  - Replace configuration file parsers with memory-safe parser. PR openembedded#725
  - Remove acountry completely, the manpage might still get installed otherwise. Issue openembedded#718

Bugfixes:

  - CMake: don't overwrite global required libraries/definitions/includes which
    could cause build errors for projects chain building c-ares. Issue openembedded#729
  - On some platforms, netinet6/in6.h is not included by netinet/in.h
    and needs to be included separately. PR openembedded#728
  - Fix a potential memory leak in ares_init(). Issue openembedded#724
  - Some platforms don't have the isascii() function. Implement as a macro. PR openembedded#721
  - CMake: Fix Chain building if CMAKE runtime paths not set
  - NDots configuration should allow a value of zero. PR openembedded#735

Signed-off-by: Jason Schonberg <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kraj @large2021