Re: [RFS] ruby-oj 3.13.23.2

[Pirate Praveen]

On 26/12/22 10:18 PM, Vinay wrote:



On 26/12/22 16:20, Pirate Praveen wrote:



On 24/12/22 11:37 PM, Vinay wrote:


Hello Team,
I have updated ruby-oj(3.13.23.1  -> 3.13.23.2 ) .The updated 
package is available at [0].



The versions here are 3.13.23-1  -> 3.13.23-2


ruby-oj (3.13.23 ) is required for the new gem of gitlab oj-introspect


Reason for sending an RFS mail is that the package ruby-oj's header 
files were required by oj-introspect.


[0] https://salsa.debian.org/ruby-team/ruby-oj

Uploaded, thanks for your contribution! From next time, it is good to 
have changelog changes committed separately to easy branching and 
merging when backporting, fasttracking etc.


Okay will commit the changelog changes separately,but i also few 
changes to be done to d/rules again,as discussed here


https://lists.debian.org/debian-ruby/2022/12/msg00043.html
to install the headers in ruby specific directory instead of /usr/include

execute_before_dh_auto_install:
    for r in $$(ruby -rruby_debian_dev -e 'puts 
RubyDebianDev::SUPPORTED_RUBY_VERSIONS.values'); do \
    dh_install ext/oj/oj.h $$($$r -e 'puts 
RbConfig::CONFIG["vendorhdrdir"]'); \
    dh_install ext/oj/cache8.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/rxclass.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/parser.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/buf.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/usual.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \

    done


Should it be as new changelog entry(3.13.23.3) or same entry 3.13.23.2


I have uploaded already, so it should be a new version (3.13.23-3 not 
3.13.23.3)

Re: [RFS] ruby-oj 3.13.23.2

[Vinay]

On 26/12/22 16:20, Pirate Praveen wrote:



On 24/12/22 11:37 PM, Vinay wrote:


Hello Team,
I have updated ruby-oj(3.13.23.1  -> 3.13.23.2 ) .The updated package 
is available at [0].



The versions here are 3.13.23-1  -> 3.13.23-2


ruby-oj (3.13.23 ) is required for the new gem of gitlab oj-introspect


Reason for sending an RFS mail is that the package ruby-oj's header 
files were required by oj-introspect.


[0] https://salsa.debian.org/ruby-team/ruby-oj

Uploaded, thanks for your contribution! From next time, it is good to 
have changelog changes committed separately to easy branching and 
merging when backporting, fasttracking etc.


Okay will commit the changelog changes separately,but i also few changes 
to be done to d/rules again,as discussed here


https://lists.debian.org/debian-ruby/2022/12/msg00043.html
to install the headers in ruby specific directory instead of /usr/include

execute_before_dh_auto_install:
    for r in $$(ruby -rruby_debian_dev -e 'puts 
RubyDebianDev::SUPPORTED_RUBY_VERSIONS.values'); do \
    dh_install ext/oj/oj.h $$($$r -e 'puts 
RbConfig::CONFIG["vendorhdrdir"]'); \
    dh_install ext/oj/cache8.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/rxclass.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/parser.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/buf.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \
    dh_install ext/oj/usual.h $$($$r -e 'puts 
RbConfig::CONFIG["rubyhdrdir"]'); \

    done


Should it be as new changelog entry(3.13.23.3) or same entry 3.13.23.2

Re: [RFS] ruby-oj 3.13.23.2

[Pirate Praveen]

On 24/12/22 11:37 PM, Vinay wrote:


Hello Team,
I have updated ruby-oj(3.13.23.1  -> 3.13.23.2 ) .The updated package 
is available at [0].



The versions here are 3.13.23-1  -> 3.13.23-2


ruby-oj (3.13.23 ) is required for the new gem of gitlab oj-introspect


Reason for sending an RFS mail is that the package ruby-oj's header 
files were required by oj-introspect.


[0] https://salsa.debian.org/ruby-team/ruby-oj

Uploaded, thanks for your contribution! From next time, it is good to 
have changelog changes committed separately to easy branching and 
merging when backporting, fasttracking etc.




Thank You
Vinay Keshava