Activity
From 09/12/2021 to 09/18/2021
09/18/2021
-
06:51 PM Bug #17882: bootstraptest/test_ractor.rb:224 segfaults on Cygwin
- This no longer happens as of commit:4cc44bd819, though not bisected. Thanks.
-
04:08 PM Bug #17885 (Closed): require_relative and require should be compatible with each other when symlinked files are used
- Applied in changeset commit:git|ddb85c5d2bdf75a83eb163856508691a7436b446.
----------
Do not load file with same realpath twice when requiring
This fixes issues with paths being loaded twice in certain cases
when symlinks are used.
It ... - 04:05 PM Revision f9fd04ff (git): * 2021-09-19 [ci skip]
-
- This fixes issues with paths being loaded twice in certain cases
when symlinks are used.
It took me multiple attempts to get this working. My original
attempt tried to convert paths to realpaths before adding them
to $LOADED_FEATURES. ... -
02:21 PM Misc #18175: Propose Jean Boussier (@byroot) as a core committer
- Accepted.
Matz.
-
01:45 PM Bug #18141: Marshal load with proc yield objects before they are fully initialized
- I made a followup patch: https://github.com/ruby/ruby/pull/4866
It now handle similar bugs with `Array`, `Hash` and other mutable objects. It also handle circular objects. -
07:34 AM Bug #18141: Marshal load with proc yield objects before they are fully initialized
- Hello byroot,
Thank you for the investigation about the issue. Yes, the patch with only the bug fix is very helpful to maintain stable branches. -
12:53 PM Bug #18178 (Closed): RubyVM::AST.of(method_defined_in_C) Segmentation fault
- Applied in changeset commit:git|ed9d9cee76ee941c504ccea33c70c0eb5d825813.
----------
ast.c: AST.of checks if a given method object is defined in C
[Bug #18178] -
10:05 AM Bug #18178 (Closed): RubyVM::AST.of(method_defined_in_C) Segmentation fault
- `RubyVM::AbstractSyntaxTree.of` segfault when it receives a method defined in C.
```console
$ ruby -ve 'RubyVM::AbstractSyntaxTree.of(String.instance_method(:gsub))'
ruby 3.1.0dev (2021-09-17T11:46:18Z master b45fe48fbb) [x86_64-... -
-
- [Bug #18178]
-
12:27 PM Revision 1c07c982 (git): Enhanced RDoc for Range (#4847)
- Treated:
#to_s
#inspect
#===
#include?
#cover?
#count - 09:04 AM Revision 593cc4ca (git): * 2021-09-18 [ci skip]
-
09:03 AM Revision ec4ecb6d (git): Include the invalid value in invalid_utc_offset error
-
- ruby_3_0 3e61556191ea7e6586ffb995f03e6d2631ec6fb0 merged revision(s) 99d8c4832a7133ca52578d015e3ddcfd94820f4a.
-
- ruby_3_0 7b55ff21b52e818819b698326f80158643175645 merged revision(s) 564ccd095a9d7fbe869031dbf666d61dadfdcb03.
-
- ruby_3_0 e6e25b794d8db52e1df85a02f28846ad7eb82d49 merged revision(s) 13939d61b4b69bd109c5f41303c79868d639fa44.
-
- Additionally backported bb84c75001f1bf13b4b2a12db8f4420e76a3ea03 into ruby_3_0 at 98ac62de5cb03efec0fb32684c61c0d4df692e5a.
-
- Revert "Force recycle intermediate collection in Hash#transform_keys!
[Bug #17735]"
This reverts commit 522d4cd32f7727886f4fcbc28ed29c08d361ee20.
---
hash.c | 1 -
1 file changed, 1 deletion(-) -
- b8386f7f7f6d7a7d76481e02d389d0f5211f0f2c was already backported into ruby_3_0 branch at 80c180df769016d58663218de2687613841864f3.
-
- Preserve the encoding of the argument in IndexError [Bug #18160]
---
re.c | 20 ++++++++++----------
test/ruby/test_regexp.rb | 7 ++++++-
2 files changed, 16 insertions(+), 11 delet... -
- [ruby/strscan] Fix segmentation fault of `StringScanner#charpos` when
`String#byteslice` returns non string value [Bug #17756] (#20)
https://github.com/ruby/strscan/commit/92961cde2b
---
ext/strscan/str... -
- Check if closed after each yield [Bug #17661]
---
io.c | 4 +++-
test/ruby/test_io.rb | 36 ++++++++++++++++++++++++++++++++++++
2 files changed, 39 insertions(+), 1 deletion(-) -
04:48 AM Bug #18177 (Closed): An error occurs when running RSpec with Ruby 3.1.0-dev
- ## Reproduction context
The following code causes an `ArgumentError` error when using Ruby 3.1.0-dev.
```ruby
# foo_spec.rb
RSpec.shared_context 'config', :config do
end
RSpec.describe 'foo', :config do
end
```
RSpec ver... -
02:48 AM Bug #18169: Local copies of gemified libraries are being released out of sync with their gems
- Dan0042 (Daniel DeLorme) wrote in #note-10:
> If I understand correctly, you mean that it's hard/impossible to test the gemified libs via CI unless they are part of the ruby monorepo? Is that the main difficulty?
Yes. Ex. "ruby/setup... -
12:22 AM Feature #18151: Incorrect Resolv result when DNS server is unreachable
- The current behavior seems to be expected (the code explicitly rescues the internal ResolvError), so I would consider this a feature request and not a bug. I agree with you that it would be nice to be able to differentiate the cases. I...
09/17/2021
-
09:51 PM Feature #18159: Integrate functionality of syntax_suggest gem into Ruby
- > We want to make sure there's no (or very low) overhead for correct programs. So possibly load dead_end code only when the error occurs, because it's not needed otherwise.
Hooking into SyntaxError#message should make it zero overhead... -
08:59 AM Feature #18159: Integrate functionality of syntax_suggest gem into Ruby
- Some additional comments during the meeting:
- We want to make sure there's no (or very low) overhead for correct programs. So possibly load dead_end code only when the error occurs, because it's not needed otherwise.
- What happen... -
- > ko1 (Koichi Sasada) and knu (Akinori MUSHA) were concerned about whether the heuristic algorithm was mature enough. If dead_end prints a misguided summary, it may rather confuse users.
I certainly understand and agree. If you (or an... -
- I've submitted a pull request to have `Pathname#each_entry` return Enumerator: https://github.com/ruby/pathname/pull/10
-
- I implemented a patch https://github.com/ruby/ruby/pull/4859
NB: it does include some extra fixes for https://bugs.ruby-lang.org/issues/18141#change-93742, because the implementation wouldn't have been possible without. -
02:44 PM Bug #18169: Local copies of gemified libraries are being released out of sync with their gems
- hsbt (Hiroshi SHIBATA) wrote in #note-9:
> Because you are not develop CRuby. We need to test EVERY commits of CRuby with default gems. If CRuby or stdlib broke the test/CI,
> ...
If I understand correctly, you mean that it's hard/impo... -
- >Why not make the changes to the gem itself and be certain that all changes are already in the canonical repository?
> ...
Because you are not develop CRuby. We need to test EVERY commits of CRuby with default gems. If CRuby or stdlib b... -
- So while working on https://bugs.ruby-lang.org/issues/18148, I discovered that many other types of objects are impacted.
Just a few examples:
```ruby
def round_trip(obj, proc = ->(o) { o.freeze })
Marshal.load(Marshal.dump(obj)... - 11:46 AM Revision b45fe48f (git): [rubygems/rubygems] Trigger install command by default on remove
- Closes https://github.com/rubygems/rubygems/issues/4889
https://github.com/rubygems/rubygems/commit/2b1754479c -
- Applied in changeset commit:git|178ee1e801acb33d13b3e8a630f6ca4926c68fbc.
----------
Already initialized modules cannot be replaced [Bug #17048] -
09:48 AM Bug #17429: Prohibit include/prepend in refinement modules
- shugo (Shugo Maeda) wrote in #note-31:
> mame (Yusuke Endoh) wrote in #note-30:
> ...
I've created an issue on the project: <https://github.com/square/refinement/issues/71>.
By gem-codesearch, I've found another gem which defines ::... -
- mame (Yusuke Endoh) wrote in #note-30:
> BTW, the change seems to add a top-level new constant `::Refinement`. I'm not against the addition, but unsure about the impact. Is it okay?
I found a gem named refinement....
https://githu... -
- * [Feature #18176] Make Coverage suspendable (mame)
* I'd like to add `Coverage.suspend` and `Coverage.resume`. Any opinion?
* [Bug #18170] `Exception#inspect` should not include newlines (mame)
* I've created a PR for `StandardError... -
- Created a PR: https://github.com/ruby/ruby/pull/4857
As expected, I had to modify some tests. Is this acceptable? -
- +2
-
03:17 AM Misc #18175: Propose Jean Boussier (@byroot) as a core committer
- +1
-
- I'd like to add `Coverage.suspend`, `Coverage.resume`, and some methods.
## Synopsis
```
1: # target.rb
2: def foo
3: :foo
4: end
5:
6: def bar
7: :bar
8: end
9:
10: def baz
11: :baz
12: end
```
```
... -
06:09 AM Bug #14479 (Rejected): Exceptions raised from a :call tracepoint can sometimes be "rescued" inside the method
- I want to reject this issue because of the following reasons:
* TracePoint block shouldn't raise an exception. TracePoint should not hurt non-hook (99.99..% case) execution if possible. I don't think this difference is not a matter.
... -
-
-
-
01:03 AM Revision 5a048829 (git): Add WB_PROTECTED to mutexes
- mutex_mark is (basically) NULL, so we don't have any references to mark.
This means we should safely be able to mark Mutex as WB_PROTECTED
without changing anything else. -
- `match_ary_aref` may return an `Array` of substrings when `idx` is a `Range`.
09/16/2021
-
- 11:39 PM Revision 8450b6a9 (git): * 2021-09-17 [ci skip]
-
-
09:00 PM Bug #17048: Calling initialize_copy on live modules leads to crashes
- Here a repro that that always crashes for me on cbbda3e:
```ruby
mod = Module.new { define_method(:foo) {:first} }
klass = Class.new { include mod }
instance = klass.new
p instance.foo
new_mod = Module.new { define_method(:foo) {... -
- nobu (Nobuyoshi Nakada) wrote in #note-16:
> As far as I tested it with `master` slightly, it didn't reproduce.
> ...
It still happens in my environment. I tested with 1000 iterations, and it hung on the 14th iteration:
```
$ for ... -
- As far as I tested it with `master` slightly, it didn't reproduce.
Does it still happen, and how occational? -
05:30 PM Misc #18175 (Closed): Propose Jean Boussier (@byroot) as a core committer
- Hi,
I would like to propose adding Jean Boussier (@byroot) as a committer. For years Jean has been doing great work with performance / memory optimizations, reporting and fixing bugs, as well as contributing features that really help ... -
-
- ko1 (Koichi Sasada) wrote in #note-2:
> mame-san also proposed `Fiber.handle_interrupt` for fiber local handler to set fiber local setting.
No I didn't propose it actually. In today's dev meeting, we discussed that there are a case w... -
- I understand some case it is confusing, but if Fiber is used to represent the data structure like Enumerator, Thread-wide mask is needed.
Could you give us the example which should be fiber local?
mame-san also proposed `Fiber.handle... -
04:02 PM Feature #18172: MatchData#sublen to return the length of a substring
- nobu (Nobuyoshi Nakada) wrote in #note-4:
> Hanmac (Hans Mackowiak) wrote in #note-3:
> ...
i just wonder why not use the functions there like done in the other method?
``` c
if (FIXNUM_P(idx)) {
return rb_reg_nth_match(FIX2INT(... -
- Hanmac (Hans Mackowiak) wrote in #note-3:
> @nobu, isn't your `MatchData#match` the same as `MatchData#[]` ?
Similar, but `#match` accepts only single index/name, but not a range or an optional length. -
- @nobu, isn't your `MatchData#match` the same as `MatchData#[]` ?
-
- Applied in changeset commit:git|09d724e6f846b4e53e8571d41ca7d3055d732d9f.
----------
[Feature #18172] Add MatchData#match
The method to return the single matched substring corresponding to
the given argument. -
- In Ruby C source code, we use names like `sublen` but Ruby methods tend to be fully spelled. So I propose `MatchData#match(n)` and `MatchData#match_length(n)`.
Matz.
-
- There are many code taking the length of a substring matched by `Regexp`.
For instance, in rdoc/markup/attribute_manager.rb:
```ruby
attr_updated = attrs.set_attrs($`.length + $1.length + $2.length, $3.length, attr)
if at... -
- # The next dev meeting
**Date: 2021/10/21 13:00-17:00 +09:00**
Place/Sign-up/Agenda/Log: https://github.com/ruby/dev-meeting-log/blob/master/DevelopersMeeting20211021Japan.md
- Dev meeting *IS NOT* a decision-making place. All dec... -
-
- In today's dev meeting, we discussed this. Basically, no one was opposed to the concept. But some concerns were discussed.
@ko1 and @knu were concerned about whether the heuristic algorithm was mature enough. If dead_end prints a misg... -
02:50 PM Bug #18169: Local copies of gemified libraries are being released out of sync with their gems
- > We should backport this changes to the upstream repo in github.
Why not make the changes to the gem itself and be certain that all changes are already in the canonical repository?
If it is a problem of testing in CRuby, you can r... -
- We are sorry to confuse about the versioning policy for the default gems in our transition phase.
I only maintain the version of the default gems with the new release version of Ruby like 2.6.0.
I forgot them with stable version releas... -
- An alternative might be to check out the repositories at a specific tag and copy them in place, rather than going to the gems.
-
- I am available to help make this happen or describe how we are doing it in JRuby. The short description is just that we install to a temporary location and copy everything in place as part of the build. However, we do that by bootstrappi...
-
-
02:15 PM Feature #18136: take_while_after
- @mame
> In other words, your use case examples looked not so frequent to make it built-in.
Soooo... Let me summarize :)
In defense of the idea, I provided:
1. **Two** big domains: API clients with pagination ("stop after curr... -
- zverok (Victor Shepelev) wrote in #note-11:
> > The listed use cases are not so clearly convincing. For the Example 2, "can_continue" field often contains a URL to receive the subsequent results (like `"can_continue": "http://api.exampl... -
- Then what about `take_till`?
```ruby
Enumerator.produce(Date.today, &:succ).take_till(&:monday?).each { schedule_job(_1) }
``` -
- @knu
> feels to me like `drop_after`
Hmm, this actually sounds like a good name.
> ...
Of course.
The first case of my initial ticket (take tokens till the token that ends sentence)
```ruby
sentence = tokens.take_while_aft... -
- In the dev meeting, we discussed this proposal in more than a hour, and reached no conclusion.
* The listed use cases are not so clearly convincing. For the Example 2, "can_continue" field often contains a URL to receive the subsequen... -
04:56 AM Feature #18136: take_while_after
- I want to see use cases without `lazy`. If typical uses cases you could think of were always with `lazy`, then a lazy-minded API would be what you'd need in the first place.
-
- This behavior of `take_upto` feels to me like `drop_after`. 🤔
-
- ufuk (Ufuk Kayserilioglu) wrote in #note-5:
> I don't want to detract from the content of the proposal, nor do I want to bikeshed the name, but personally, `take_while_after` does not convey the meaning of this behaviour to me. Can I su... -
- > Could you please create a patch for your proposal?
That was my intent, it might take me a couple days though. -
- @nobu created a ticket #18171 for `Hash#compare_by_identity` issue. It is not so related to this issue, anyway.
@byroot Could you please create a patch for your proposal? -
- Seems reasonable. Accepted.
@nobu might have concerns regarding `Hash#compare_by_identity`.
Matz.
-
- BTW, the change seems to add a top-level new constant `::Refinement`. I'm not against the addition, but unsure about the impact. Is it okay?
-
- In today's dev meeting, matz accepted the concept, but wanted to take some time to consider the name `import`.
-
-
- The method to return the length of the matched substring
corresponding to the given argument. -
- The method to return the single matched substring corresponding to
the given argument. -
- Applied in changeset commit:git|ddb32e66160ab50849419ef7c7ac584913b79c34.
----------
[Bug #18173] Update loaded_features_index
If $LOADED_FEATURES is changed in the just required file, also the
index table needs to be updated before lo... -
- In general, the feature name registered in `$LOADED_FEATURES` (e.g. "enumerator.so", "thread.rb", and so on) is expected not to be loaded again.
But, as @ko1 reported, when a feature name is added to the variable during another file i... -
10:16 AM Bug #18170: Exception#inspect should not include newlines
- Thanks @mame.
-
- Matz said "give it a try". I'll create a PR
-
- Is this intentional?
```
p StandardError.new("foo\nbar")
#=>
# #<StandardError: foo
# bar>
```
I expect `#inspect` returns a one-line string. How about returning `#<StandardError: "foo\nbar">` or something?
Recently, multi-... -
- If $LOADED_FEATURES is changed in the just required file, also the
index table needs to be updated before loaded_features_snapshot is
reset. If the snapshot is reset without updating the table, the
name of the added feature will not be ... -
- ...Which is that `{ symbol: }` verbosely means `{ symbol: binding.local_variable_defined?(:symbol) ? binding.local_variable_get(:symbol) : __send__(:symbol) }` with no exception, no matter if the symbol is `if`, `self`, `fork`, `return` ...
-
- Matz accepted the current behavior at [DevelopersMeeting20210916Japan](https://bugs.ruby-lang.org/issues/18122)
-
- matz (Yukihiro Matsumoto) wrote in #note-16:
> I assumed the value should be a local variable.
I also assumed the same thing, but after getting over my initial surprise I found this way is quite nice, very ruby-ish. A bit like learni... -
-
-
- OK, accepted.
Matz.
-
- The methods may be overridden in the subclass. The assumption in this report seems to be too much.
Rejected.
Matz.
-
- Is this a bug?
```ruby
h = {}.compare_by_identity
h["a"+"1"] = "1"
h["a"+"1"] = "2"
p h #=> {"a1"=>"1", "a1"=>"2"}
p Marshal.load(Marshal.dump(h)) #=> {"a1"=>"2"}
``` -
- I reject this proposal, for several reasons:
* it's very easy to implement the method using existing `GC.stress` and `begin ... ensure` by yourself
* if it's built-in, users may expect too much (thread-safety is one of them)
Matz.
-
04:21 AM Revision 68bd5626 (git): Remove unneeded ruby_reset_leap_second_info function declaration (#4824)
-
- In the end I agree that `touch` is enough and `ensure_parent_dir_exists` is unnecessary (even with a shorter name). Even though creating the file via "touch" is kinda redundant before `open('a')` it's not really a problem either.
sch...
09/15/2021
-
-
- I am unable to edit the description, but I wanted to make clear that the two proposed solutions are mutually exclusive, and I **strongly** recommend that the gem sources be removed from the CRuby repository rather than auditing them at r...
-
- The CRuby codebase includes a number of libraries that have been gemified, more and more with each release. Unfortunately, these libraries are continually out of sync with both their home repositories and their released gems.
The prob... -
-
- I've made a draft PR with a working spike of getting this into Ruby. https://github.com/ruby/ruby/pull/4845 it should be good enough for discussion. I used the same interface you suggested here. It was very easy.
The only limitation ... -
-
- > Shouldn't this method take keyword arguments that FileUtils.touch accepts?
I looked into it. Of the existing pathnames that delegate to FileUtils, only one supports kwargs and it does not support all of them, just one:
```
d... - 06:37 PM Revision 67e3727d (git): * 2021-09-16 [ci skip]
-
-
-
- We can close this due to addition of error_highlight in ruby https://github.com/ruby/ruby/tree/db51bcada408f8b0b93a11d41f8e4222436bd9f2/lib/error_highlight
- 01:59 PM Revision db51bcad (git): Fix total_freed_objects for invalidated pages
- When the object is moved back into the T_MOVED, the flags of the T_MOVED
is not copied, so the FL_FROM_FREELIST flag is lost. This causes
total_freed_objects to always be incremented. - 01:00 PM Revision a65ac2d6 (git): Don't overwrite free_slots count during sweeping
- gc_compact_finish may invalidate pages, which may move objects from this
page to other pages, which updates the free_slots of this page. - 01:00 PM Revision e624d0d2 (git): Update the free_slots count of the original page
- When invalidating a page during compaction, the free_slots count should
be updated for the page of the object and not the page of the forwarding
address (since the object gets moved back to the forwarding address). -
- duerst (Martin Dürst) wrote in #note-27:
> I don't think using `if` as the name of a local variable is a good idea, and I don't think the speed of bad ideas should concern us too much.
As zverok stated, a keyword such as `if` is used... -
- @duerst
> I don't think using `if` as the name of a local variable is a good idea,
It is good (and widely used, BTW) name for a method parameter, in contexts like
```ruby
validate :foo, if: :something?
```
I don't see how it is... -
- shugo (Shugo Maeda) wrote in #note-26:
> duerst (Martin Dürst) wrote in #note-25:
> ...
I don't think using `if` as the name of a local variable is a good idea, and I don't think the speed of bad ideas should concern us too much. -
- duerst (Martin Dürst) wrote in #note-25:
> Ah, so `{if:}` means something close to `{if: local_variable_get(:if)}` and '{self:}` means something close to `{self: local_variable_get(:self)}` (and NOT `{self: self}`).
Yes.
Technicall... -
- shugo (Shugo Maeda) wrote in #note-24:
> I meant that keywords are allowed as local variable or method names.
> ...
Ah, so `{if:}` means something close to `{if: local_variable_get(:if)}` and '{self:}` means something close to `{self... -
- shugo (Shugo Maeda) wrote in #note-22:
> except that keywords are allowed.
I meant that keywords are allowed as local variable or method names.
For example, `{if:}` is not a syntax error and `{self:}` doesn't access the receiver but... -
- Note that constants are also allowed:
```ruby
X = 1
p(X:) #=> {:X=>1}
``` -
- matz (Yukihiro Matsumoto) wrote in #note-16:
> I assumed the value should be a local variable. The merged patch calls the method when the local variable is not defined.
> ...
I believe a method should be called when a local variable is... -
07:55 AM Revision 92ac1721 (git): [ruby/racc] gemspec: Link to https in homepage [ci skip]
- https://github.com/ruby/racc/commit/96075f7720
-
- The release of Unicode Version 14.0.0 has officially been announced today, see
https://home.unicode.org/announcing-the-unicode-standard-version-14-0/.
The overall summary of the version is at http://www.unicode.org/versions/Unicode14.0... - 07:20 AM Revision 645acf57 (git): [ruby/racc] Prefer __dir__ over File.dirname(__FILE__)
- https://github.com/ruby/racc/commit/c49df40658
-
-
05:29 AM Feature #18168 (Open): Add ActiveSupport deep_transform_values to Ruby
- I think since transform_values is a part of Ruby, it makes sense to support the nested version of transform values
-
- * As the "doc/" prefix is specified by the `--page-dir` option,
remove from the rdoc references.
* Refer to the original .rdoc instead of the converted .html. -
12:25 AM Revision 63ab2fed (git): include/ruby/atomic.h: add doxygen
- Must not be a bad idea to improve documents. [ci skip]
-
- applied doxygen -g for Doxygen 1.9.3.
09/14/2021
-
-
- https://github.com/ruby/racc/commit/1f8b05b298
-
- https://github.com/ruby/logger/commit/2fd5401ad6
-
- This gem exposes 0 executables.
https://github.com/ruby/logger/commit/fbd9315393 -
-
-
- Ref: https://bugs.ruby-lang.org/issues/18141
-
10:56 PM Feature #18020: Introduce `IO::Buffer` for fiber scheduler.
- +1 that being said, I have no problem with augmenting IO::Buffer to map into a memory view interface.
``` ruby
f = File.open(FILENAME,'rb')
bytearray = ByteArray.new(File.size(FILENAME)) # ByteArray implements memoryview and is mut... -
- Treated:
#size
#to_a
#each
#begin
#end
#first
#last -
- @Hanmac is correct. I think you also need to manually set `create_additions: false`, otherwise, an ArgumentError is raised:
```ruby
JSON.load(%( {"a": 1, "b": 2} ), nil, symbolize_names: true, create_additions: false)
# => {:a=>1, ... -
- The Problem lies in the documentation of the load method:
``
JSON.load(source, proc = nil, options = {}) -> object
``
because it isn't coded for newer ruby using `**options` there,
for you to make it work you need:
`JSON.load( ... -
01:03 PM Bug #18167 (Closed): JSON.load doesn't symbolize names
- As per the documentation the JSON#load method should accept _**and use**_ the same parsing options as the JSON#parse one.
Obviously this is not the case:
```
$ ./json_parse_vs_load.rb
JSON.load, no symbolize => OK
{"a"=>1, "b"=>... -
-
02:26 PM Bug #18166: Windows Ripper test-all failures? bison package?
- Fixed by @nobu in [fa05697e4832](https://github.com/ruby/ruby/commit/fa05697e4832), both ruby-loco mingw & mswin also pass.
Thanks again. Ok to close.
-
- FYI, it is known that Bison 3.8.1 breaks build of the latest "still" release of LibreOffice (7.1.6.2): https://bugs.gentoo.org/812923
-
- Windows builds recently failed, and it's odd. The bison package did change from 3.7.6 to 3.8.1...
- 09:08 PM Revision aa18f645 (git): * 2021-09-15 [ci skip]
-
- This PR creates doc/bsearch.rdoc to provide common documentation for bsearch in Array and Range.
-
03:37 PM Bug #18165: resolv: IPv6 link local addresses do not accept dash as a valid zone_id character
- jeremyevans0 (Jeremy Evans) wrote in #note-3:
> This issue should be fixed by https://github.com/ruby/resolv/pull/2, which is still waiting for review.
Thanks for the link Jeremy. I tried to find a similar bug here but I did not look... -
- This issue should be fixed by https://github.com/ruby/resolv/pull/2, which is still waiting for review.
-
-
- Applied in changeset commit:git|57d315c937e79199af2b77f21f5eecaca85ffac8.
----------
Handle overwriting Object::ENV in spawn
Instead of looking for Object::ENV (which can be overwritten),
directly look for the envtbl variable. As that... -
- Instead of looking for Object::ENV (which can be overwritten),
directly look for the envtbl variable. As that is static in hash.c,
and the lookup code is in process.c, add a couple non-static
functions that will return envtbl (or envtbl... -
01:28 PM Revision 616d6717 (git): [ruby/reline] Remove unused using method
- follow #347
https://github.com/ruby/reline/commit/37453a83a3 -
- 522d4cd32f7727886f4fcbc28ed29c08d361ee20 was reverted at bb84c75001f1bf13b4b2a12db8f4420e76a3ea03. Should I backport it into ruby_3_0?
-
- 07:04 AM Revision 6031f426 (git): Update bundled_gems at 2021-09-14
-
- > When freeze:true is specified, only returned value from proc will be frozen?
It could be either really, but I think the proc should be called with the objects already frozen, mostly to allow deserializing strings with `rb_interned_s... -
- `Marshal.load()` accepts `proc` which can manipulate the loaded object:
```ruby
str = Marshal.dump(["a", 1, 10 ** 10, 1.0, :foo])
p Marshal.load(str, proc {|obj| [obj]})
#=> [[["a"], [1], [10000000000], [1.0], [:foo]]]
```
When... -
- It would be helpful for us if you cloud add a simple code example for a feature proposal.
```ruby=
dump = Marshal.dump(["foo", "bar", "baz"])
ary = Marshal.load(dump, freeze: true)
p ary #=> ["foo", "bar", "baz"]
p ary.froze... -
- shyouhei (Shyouhei Urabe) wrote in #note-11:
> mame (Yusuke Endoh) wrote in #note-10:
> ...
Please create a ticket. Alternatively, you can propose another good way to create an agenda markdown automatically. -
- mame (Yusuke Endoh) wrote in #note-10:
> @shyouhei @hsbt please follow the comment format. My dev-meeting setup script ignores your topics.
How? Mine is not about a BTS ticket. -
- @shyouhei @hsbt please follow the comment format. My dev-meeting setup script ignores your topics.
-
- Shouldn't this method take keyword arguments that FileUtils.touch accepts?
-
-
-
05:53 AM Revision bb84c750 (git): Revert "Force recycle intermediate collection in Hash#transform_keys! [Bug #17735]"
- This reverts commit 522d4cd32f7727886f4fcbc28ed29c08d361ee20.
-
- The advantage of this `RactorLocalSingleton` is, if a library uses singleton library and it is okay to make an instance per Ractor (memo-usage, for example), it is easy to support ractors.
-
- code: https://github.com/ruby/singleton/pull/4/files
-
- ko1 agreed to keep the current behavior, so I close this issue.
09/13/2021
-
- The mentioned patch is attached.
-
- The support for IPv6 link local addresses was added to resolv [here](https://bugs.ruby-lang.org/projects/ruby-master/repository/git/revisions/2f12af42f7f26d570219b87a89294532a86a8ae2).
However, it does not take into account a `zone_id... -
08:39 PM Revision 881136d2 (git): [ruby/reline] The behaviour of Alt+key on Windows is changed
- ref. https://github.com/ruby/reline/commit/dc2cf90fa6
https://github.com/ruby/reline/commit/4d1518aafc -
-
- jeremyevans0 (Jeremy Evans) wrote in #note-4:
> Fryguy (Jason Frey) wrote in #note-3:
> ...
I submitted a pull request that takes this approach to fixing the issue: https://github.com/ruby/ruby/pull/4834 -
- Fryguy (Jason Frey) wrote in #note-3:
> Speculation ahead, but I'm at the point where I don't understand the Ruby C code anymore...but I think what's happening is, roughly:
> ...
I agree, that is a bug. It should not be looking for `O... -
05:01 PM Bug #18164: Segfault after spawn when using modified ENV
- Speculation ahead, but I'm at the point where I don't understand the Ruby C code anymore...but I think what's happening is, roughly:
https://github.com/ruby/ruby/blob/ebad1e829316de48f212cd57f88639fa5ac55ee4/process.c#L2980-L2984
```ru... -
- This seems to not segfault
```
ENV = {}
spawn({}, "true", :unsetenv_others => true)
ENV.replace({})
```
which leads me to believe that these lines are in play: https://github.com/ruby/ruby/blob/ebad1e829316de48f212cd57f88639fa5... -
04:38 PM Bug #18164: Segfault after spawn when using modified ENV
- On my Mac (Big Sur 11.5.2) using Ruby 3.0.2, I can duplicate the segfault with just the first two lines:
```
ENV = {}
spawn({}, "true")
```
However, this does NOT segfault:
```
ENV.replace({})
spawn({}, "true")
``` -
- The attached segfault.rb causes a segfault on Ruby 3.0.2 (also on 2.7.2+). This is the smallest reproducer we could get.
```ruby
ENV = {}
spawn({}, "true")
ENV.replace({})
```
You can also change the last line to `ENV.to_s` an... -
- Treated:
::new
#include_end?
#==
#eql?
#hash
#step -
- > Can you look at whether an integration similar to did_you_mean' and 'error_highlight' is easy/possible?
I've got some time on my calendar for exploration. I didn't previously budget for it so I can't guarantee how far I'll be able t... -
- * [Bug #15027] When Struct#each method is overriden Struct#select and Struct#to_a use wrong collections (jeremyevans0)
* Do we want methods like Struct#select to internally call #each (at least if #each is overridden)? (I'm against ma... -
-
- The documentation was fixed in commit:7f4e86804d426d79807cc038fe4444f7c65f5c4a, but it hasn't been backported to 3.0. I'm marking it for backporting.
-
03:35 PM Bug #18163: Unclear String documentation of <=> and casecmp methods
- > I'd expect 'B' <=> 'A' to result in -1 since 'B' is smaller.
This should've been: `since 'A' is smaller`. -
- I have a new student learning Ruby, and there are apparent inconsistencies in Ruby documentation.
The problem appears to be in the order of parameters / variables `self` and `other_string`.
<=>
https://docs.ruby-lang.org/en/3.0.0/St... - 03:37 PM Revision ebad1e82 (git): * 2021-09-14 [ci skip]
-
- Makes What's Here sections in Time and Dir link to What's Here for super and included modules (like all the others).
-
- Also observed in ArchLinux CI:
http://rubyci.s3.amazonaws.com/arch/ruby-master/log/20210913T120007Z.log.html.gz -
-
-
-
-
-
09:11 AM Feature #17355: Using same set of names in or-patterns (pattern matching with Foo(x) | Bar(x))
- > I'm confused by this code:
Actually, I'm too! The goal of this request doesn't go beyond having support for
```ruby
case [1, 2]
in [1, a] | [a, 3] then a
end
```
The `=> a` is probably a leftover, I don't recall the reason... -
- Same as Test::Unit::CoreAssertions#assert_no_memory_leak.
-
-
04:54 AM Revision 46769217 (git): Adds mixed hash value and value omission tests
- Introduces specification tests for mixed values and value omissions for
Hashes and keyword arguments, such as `{ a:, b:, c: 3 }`. -
- baweaver (Brandon Weaver) wrote in #note-19:
> * `@var:` - Would this work with instance/class/global/constant variables if they're valid symbols?
No, because we didn't change the symbol key syntax. `{ @var: @var }` is not valid, so... -
- We also discussed further with Matz and concluded that quoted keys (`{ "key": }`) are not allowed with or without interpolation. This is simply because you don't need that when any local variable or constant can be written without quota...
-
- knu (Akinori MUSHA) wrote in #note-18:
> We should allow it to call a (private) method if no variable with the name defined. We use methods in RSpec or with attr_reader that look like variables, and programmers don't necessarily distin... -
- We should allow it to call a (private) method if no variable with the name defined. We use methods in RSpec or with attr_reader that look like variables, and programmers don't necessarily distinguish between methods from variables when ...
-
- For the record: `{ "#{ str }": }` is not allowed. Matz said that it is intentional.
-
- I assumed the value should be a local variable. The merged patch calls the method when the local variable is not defined.
I doubt this is sufficient behavior. Any opinion?
Matz. -
02:33 AM Feature #18162: Shorthand method Proc#isolate to create isolated proc objects
- The related ticket is here about Proc#isolated? https://bugs.ruby-lang.org/issues/18137
-
- Currently, isolated proc objects can be created only via Ractor.make_shareable() method.
But isolated proc objects are useful for other use-cases too. So I want to propose a new method Proc#isolate to make it isolated.
For example, i... -
- This reverts commit 62db6e47b6c0fbba337649bfa468ec221d5d1001.
308183fffab43f1e111d5bb4c60f1380432966e6 is the correct solution
about this. -
- Leaked file descriptor: TestRDocGeneratorDarkfish#test_template_stylesheets: 11 : #<File:./tmp20210913-30865-7c4ey8/hoge20210913-30865-evm2uo.css>
09/12/2021
-
- I'm confused by this code:
```
case [1, 2]
in [1, a] | [a, 3] => a then a
end # duplicated variable name
```
The variable isn't used twice, it's used four times. We can ignore the last one, because this is just to give a result b... -
- It'd be nice to have this in 3.1 (no pushing).
-
- @jhawthorn said, "this will make Ruby's integer comparisons slower
globally." It looks like "binding.irb" is going to cause serious
problems in Rails applications.
https://github.com/ruby/reline/commit/ee8d6c6a82 - 08:12 PM Revision a8fe7c9e (git): coroutine enables assembly version for dragonflybsd.
-
- 03:51 PM Revision 62570904 (git): * 2021-09-13 [ci skip]
-
- https://github.com/ruby/openssl/commit/258e30b640
-
- https://github.com/ruby/openssl/commit/0f91e2a6ee
-
- Just append OpenSSL error reason to the given message string
object, which would be alreadly formatted.
Suppress -Wformat-security warning in `ossl_tsfac_create_ts`.
https://github.com/ruby/openssl/commit/11b1d8a6b8 -
- * Add `printf` format attribute to `ossl_raise`.
* Fix a format specifier in `config_load_bio`.
* Use `ASSUME` for the unreachable condition.
https://github.com/ruby/openssl/commit/41da2955db -
- The change of tool/lib/vcs.rb in commit:b8386f7f7f6d7a7d76481e02d389d0f5211f0f2c needs to be backported, to run tool/make-snapshort where "7z" command is not installed.
-
-
- Applied in changeset commit:git|99d8c4832a7133ca52578d015e3ddcfd94820f4a.
----------
Preserve the encoding of the argument in IndexError [Bug #18160] -
-
-
-
-
-
- schneems (Richard Schneeman) wrote in #note-2:
> For additional context for anyone who could not see the presentation, I uploaded my RubyKaigi talk as unlisted on YouTube https://youtu.be/DfGG77zVVR4.
This is now also available 'offi...