Project

General

Profile

Actions

Bug #8968

closed

Baseruby is required after patching configure.in

Added by vo.x (Vit Ondruch) over 10 years ago. Updated over 10 years ago.

Status:
Rejected
Target version:
-
ruby -v:
ruby 2.1.0dev (2013-09-22 trunk 43011) [x86_64-linux]
[ruby-core:57476]

Description

It sometimes happens, that during packaging Ruby, it is necessary to patch configure.in (we are doing so in Fedora, there are patches in Debian, RVM does so occasionally as well). Unfortunately, since rev 42685, if the configure.in is patched, it has newer timestamp than sizes.c and therefore sizes.c should be regenerated. For that Ruby is required. Unfortunately, miniruby is not build yet at that stage and we don't want to have other Ruby on the system, due to possible bootstrapping issues.

I can workaround it by touch sizes.c after the patch is applied, but is there change to remove/fix this dependency? Thanks.

Updated by nobu (Nobuyoshi Nakada) over 10 years ago

  • Status changed from Open to Third Party's Issue

Updated by naruse (Yui NARUSE) over 10 years ago

  • Status changed from Third Party's Issue to Rejected

You should suggest acceptable plan.
You know such way is not acceptable.

Updated by vo.x (Vit Ondruch) over 10 years ago

naruse (Yui NARUSE) wrote:

You should suggest acceptable plan.
You know such way is not acceptable.

I know, but I had to report it for the record. I stay with 'touch' for now. Hopefully other folks come later with better idea.

Updated by nobu (Nobuyoshi Nakada) over 10 years ago

Not only sizes.c, you need BASERUBY to generate some source files after touching template files.
It's your responsibility.

Updated by vo.x (Vit Ondruch) over 10 years ago

nobu (Nobuyoshi Nakada) wrote:

Not only sizes.c, you need BASERUBY to generate some source files after touching template files.
It's your responsibility.

Yes, it may happen that I may need to change some template for whatever reason, but configure.in is probably one of the first files which needs to be changed for whatever reason. For example similar patch like this one [1] is applied to Fedora's package for ages. Not sure if that makes sense to propose it upstream.

[1] http://pkgs.fedoraproject.org/cgit/ruby.git/tree/ruby-1.9.3-always-use-i386.patch

Updated by vo.x (Vit Ondruch) over 10 years ago

Thank you for fixing this.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0