Activity
From 12/13/2019 to 12/19/2019
12/19/2019
-
11:28 PM Bug #8507 (Rejected): Keyword splat does not convert arg to Hash
- As far as I understand, no one showed any actual use case that justifies adding `nil.to_h #=> {}`.
I agree with matz, it would be better to remove `nil.to_a #=> []` instead of adding `nil.to_h #=> {}` for this issue. I don't think it... -
11:18 PM Revision 2898367b (git): Warn also numbered parameter like methods
-
- Is there anyone who is interested in this? I'd be happy if anyone could look through the result and create a patch. Contribution is welcome.
@jschleus Thanks. If possible, could you apply the checker to master? In the tarballs, th... -
05:46 PM Bug #16437 (Closed): Codespell report for Ruby 2.6.5 (and 2.7.0-rc1)
- The FOSS server fossies.org offers a new feature "Source code misspelling reports":
https://fossies.org/features.html#codespell
Although such reports are normally only generated on request, as Fossies administrator I have just cre... -
07:46 PM Bug #16288: Segmentation fault with finalizers, threads
```
--- a/thread.c
+++ b/thread.c
@@ -682,7 +682,7 @@ thread_do_start(rb_thread_t *th)
else {
args_ptr = RARRAY_CONST_PTR(args);
}
-
+ rb_funcallv(NULL, idInspect, 0, 0);
th->valu...-
04:40 PM Revision 3816cd94 (git): Add `URI#open` to warning message
-
- Applied in changeset commit:git|27ddb6e99848c397a415b785315896ab50308076.
----------
Warn also numbered parameter like parameters
[Feature #16433] -
01:27 PM Feature #16433 (Closed): Proposal: Output warning if `_1` is used in parameter name
- Currently, output warning if local variable name is `_1`.
```ruby
# warning: `_1' is used as numbered parameter
_1 = 42
```
However, if parameter name is `_1`, no output warning is issued.
```ruby
# Actual behavior
# no war... - 04:37 PM Revision 844f1fad (git): * 2019-12-20 [ci skip]
-
04:37 PM Misc #16436: hash missing #last method, make it not so consistent (it has #first)
- What is this issue? Is it just a note to everyone, or are you claiming this to be a bug, or is it a feature request?
-
04:27 PM Misc #16436: hash missing #last method, make it not so consistent (it has #first)
- `Hash` doesn't have `#first` either :)
```ruby
Hash.instance_method(:first)
# => #<UnboundMethod: Hash(Enumerable)#first(*)>
# ^^^^^^^^^^
```
The `first` method existence is just because of hash being a... -
04:21 PM Misc #16436 (Open): hash missing #last method, make it not so consistent (it has #first)
- ``` ruby
(pry):main(0)> h = {x:1, y:2, z:3}
{
:x => 1,
:y => 2,
:z => 3
}
(pry):main(0)> h.first
[
:x,
1
]
(pry):main(0)> h.first.first
:x
(pry):main(0)> h.first.last
1
```
last method not exist.
... -
- [Feature #16433]
-
- The idea is obvious, but I couldn't find it discussed anywhere on tracker before. Please point me at the previous discussions if any.
```ruby
class Array
def to_proc
proc { |v| v.dig(*self) }
end
end
# Or, alternatively,... -
- Thank you for the report.
Maybe https://github.com/brianmario/mysql2/issues/1075 seems related. Anyway, the issue seems to occur in mysql2 library. Cloud you ask them first? -
- Although it is unclear without the backtrace by ruby itself, it seems caused by a wrong argument to `Mysql2::Client#connect`.
https://github.com/brianmario/mysql2/blob/785969fbceadd96d495ec9fecc67efee03d0056c/ext/mysql2/client.c#L438
T... -
02:19 PM Bug #16434 (Third Party's Issue): rails c crashed
[NOTE]
You may have encountered a bug in the Ruby interpreter or extension libraries.
Bug reports are welcome.
For details: https://www.ruby-lang.org/bugreport.html
[IMPORTANT]
Don't forget to include the Crash Report log file u...-
- It is same as the following code without irb.
```ruby
proc {
p eval("(1..10).map{_1+_1}") #=> [84, 84, 84, 84, 84, 84, 84, 84, 84, 84]
_1
}.call(42)
```
This is because the `eval` code is executed at the current context ex... -
- ## Summary
Calling `binding.irb` in a block that uses `_1` and using `_1` in `irb` will cause unintended behavior.
## Steps to reproduce
1. Use `_1` in block
2. Use `binding.irb` in block
3. Use block with `_1` in `binding.i... -
12:13 PM Feature #15306 (Closed): Generate prelude.c using miniruby
-
- prelude.c is not anymore [1], so I'm closing this specific ticket.
However, I'll be happy if the discussion is remembered for future.
[1]: https://github.com/ruby/ruby/commit/9c2807b2df14984e3c81d72a381d9a4d288b3fbe -
09:44 AM Bug #16427: Revert did_you_mean promotion to default gem.
- vo.x (Vit Ondruch) wrote:
> Nevertheless it does not address the "disk consumption" concern. I also previously forgot to mention possible increased network bandwidth required to build and distribute the cloud image.
By how much does ... -
- I agree with @zverok here, a method behaves as a lambda, and doesn't unpack arguments (except a few special methods that specifically do that).
@nobu I think we should merge your PR. Could you show an example of a Symbol#to_proc Proc ... -
- > As a symbol proc cannot know the method to be invoked, so now I think it cannot be lambda.
> ...
@nobu, I am not sure I get it right. Can you please show when it is not true?..
For as far as I can understand, there are two distinctio... -
- Just curious: How do you want to use the result of `lambda?`? Even if it returns true, we may pass an arbitrary number of arguments: `lambda {|*a| ... }`. I think that `lambda?` is useless except debugging.
-
- As a symbol proc cannot know the method to be invoked, so now I think it cannot be lambda.
In the case `:+`, it looks like a lambda, but it is not always true. -
08:30 AM Feature #15982: Remove y2racc and racc2y
- I will try to remove it at racc-1.5 in the next year.
-
- JustJosh (Joshua Stowers) wrote:
> @sawa - I 100% agree with what you are saying. I did a poor job expressing my concerns.
> ...
I understand your point. If that is a concern, then we can simply not allow the abbreviated form for `Arra... -
04:52 AM Feature #16039: Array#contains? to check if one array contains another array
- @sawa - I 100% agree with what you are saying. I did a poor job expressing my concerns.
I am nervous that singling out array arguments in the abbreviated form could result in confusion and misuse. This is not a problem for `Range#cove... -
- JustJosh (Joshua Stowers) wrote:
> I do not think we should use the name `cover?` because the types of arguments accepted by `Range#cover?` would be incompatible with this use case.
> ...
When the argument is an array, it should be und... -
- @alanwu
Please add `(alanwu)` in the next time. My agenda generator overlooks your post. -
-
-
-
12/18/2019
-
07:29 PM Bug #16427: Revert did_you_mean promotion to default gem.
- I think the issue is that the filing is about reverting the promotion to
default gems, which is different to the use case of making it (or what a
ruby user wants to use) more flexible.
The ideal scenario would be to have a fully cus... -
- rbjl (Jan Lelis) wrote:
> Then, a good configuration for the cloud could be this?
> ...
It should probably be `$ RUBYOPT=--disable-did_you_mean ruby -e "p defined? DidYouMean" # => nil` because env variables are typically easier to con... -
12:36 PM Bug #16427: Revert did_you_mean promotion to default gem.
- > But all these applications have to pay for longer startup time, higher memory and disk consumption. All these are not negligible in the age of cloud. I am pretty sure there is also performance hit during exception handling.
Then, a ... -
- mame (Yusuke Endoh) wrote:
> But it's just like saying that "ruby cannot start when I remove lib/rubygems.rb".
Speaking of which is problematic by itself. But the difference is that 99% of Ruby applications needs RubyGems for their r... -
- It does not make sense at all.
In Ruby 2.6, did_you_mean was a bundled gem, which may be uninstalled by a user. As it is not always able to require, gem_prelude.rb did `require "did_you_mean" rescue LoadError`.
In Ruby 2.7, did_you_... -
- While I agree on the situation at hand (being able to use did you mean
or disable it at the discretion of the user), I think there was a wrong
filing in the linked issue in that the suggestion of the linked in
thread was "Revert did_y... -
- I just have opened PR [1], which allows Ruby to run when did_you_mean is not available. As I previously discussed in #16427, there are cases when speed, memory, disk or network bandwidth is a concern and did_you_mean is not useful for ru...
-
- # *#cover?*
I do not think we should use the name `cover?` because the types of arguments accepted by `Range#cover?` would be incompatible with this use case.
For example:
```ruby
(1..3).cover?(2) # true
```
But if Array's implem... - 05:08 PM Revision e732e007 (git): * 2019-12-19 [ci skip]
-
05:08 PM Revision 1e88f6eb (git): Refactor free page insertion
- I am trying to fix this error:
http://ci.rvm.jp/results/trunk-gc_compact@silicon-docker/2491596
Somehow we have a page in the `free_pages` list that is full. This
commit refactors the code so that any time we add a page to the
`free... -
- Sorry, I don't think we can do anything for this report.
Did you manually build ruby and install it to /usr/local? If so, I'm unsure why chromebrew invokes it.
I presume you are using a ruby package from Linux distribution or chrome... -
- It is by design. doc/extension.rdoc says:
```
T_FIXNUM can be converted to a C integer by using the
FIX2INT() macro or FIX2LONG(). Though you have to check that the
data is really FIXNUM before using them, they are faster.
```
... -
- Closing as it is not a bug.
Now you can see unified coverage reports for Ruby's C and Ruby code in https://rubyci.org/coverage. It is measured under GitHub Actions: https://github.com/ruby/actions/blob/master/.github/workflows/covera... -
- Could you check the path of `@file1` in that file, and the filesystem of that volume?
-
- BTW preciously, I was testing against 053f78e139 where the bug had not been present yet.
-
- ~~~
$ cat /etc/os-release
NAME=Fedora
VERSION="32 (Rawhide)"
ID=fedora
VERSION_ID=32
VERSION_CODENAME=""
PLATFORM_ID="platform:f32"
PRETTY_NAME="Fedora 32 (Rawhide)"
ANSI_COLOR="0;34"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fe... -
- Note that `make runnable` makes links in `bin/`, which might be able to run if configured with `--enable-load-relative` option.
-
- Closing as it is not a bug.
A ruby binary that is built but not installed is half-baked. It is very hacky to run it without installation; it requires setting library paths, GEM_HOME, etc and etc. In the build process, the half-baked... -
- `"\x{00000020}"` も書けないわけなので、とりあえず現行の挙動は意図的であると考えます。逆に、`/\x{00000020}/` を許すなら `"\x{00000020}"` も許さねば、ってことになるのではないでしょうか。そういう仕様変更に意見はないですが、Bug ではなかろうということで閉じます。
`ONIG_SYN_OP_ESC_X_BRACE_HEX8` は実装詳細なんで、直したかったら成瀬さんの言うとおりにするか、もしくはupstreamを直... -
- minirubyだと一致するのは、rubygemsをロードしていないからのようです。
```
$ TZ=Asia/Tokyo ./bin/ruby --disable=gems -ve "ENV['TZ']='right/Asia/Tokyo';t=Time.at(2<<30);p t;p t.utc"
ruby 2.7.0dev (2019-12-18T05:53:19Z master efbc0d499c) [x86_64-linux]
2038-01-... -
08:34 AM Bug #15177: 起動した後にTZを変更するとTime#utcがずれることがある
- @naruse これ何がどうFeedbackなんですか？ここまでのやりとりだと @znz は要求されている情報を既にもれなく答えているとしか読めませんが。
-
01:41 PM Bug #16159: rubyspec about time fails in Asia/Kuala_Lumpur timezone
- ruby_2_6 r67836 merged revision(s) d6a2bce64a7fa1099e507e1d36b5f1533f42f60f,c687be4bc01c9ce52ea990945d9304d6fe59fe9b.
-
- time.c (find_time_t): fix round-to-zero bug
`find_time_t` did not work correctly for year older than the Epoch
because it used C's integer division (which rounds negative to zero).
For example, `TIme.new(1933)` ... -
- Closing as it is by design. If you want the change, please open a feature request ticket with a use case.
-
- Following log indicates that the SEGV happens inside of ffi.
> -- Control frame information -----------------------------------------------
> ...
and
> 759d3cd000-759d3fa000 r-xp 00000000 00:26 266588 /home... -
06:50 AM Feature #16428: Add Array#uniq?, Enumerable#uniq?
- Following a suggestion of `Enumerable#uniq?`, I also added `Enumerable#uniq?` to my patch.
`Array#uniq?` is left because it is faster than `Enumerable#uniq?`. -
05:02 AM Feature #16428: Add Array#uniq?, Enumerable#uniq?
- I seem to member that many years ago, I made the same proposal, and Nobu created a patch, but unfortunately, I didn't find any traces anymore on this tracker or in my mail.
Anyway, I support this proposal. It's definitely an useful fu... -
-
05:38 AM Bug #16384: Cannot cross-compile 2.7: undefined method `each_builtin' for RubyVM:Class (NoMethodError)
- I confirmed that the problem was fixed in cf948a3d3f and 2.7.0-rc1. Thank you for the fix!
-
04:49 AM Bug #16384 (Closed): Cannot cross-compile 2.7: undefined method `each_builtin' for RubyVM:Class (NoMethodError)
- could you try on current master? It was fixed.
40026a408d -
- Nobody uses it any longer.
-
-
- Errno::EROFS may occur when the source tree is placed in a
read-only filesystem. -
- Hangs up only in ruby/ruby on Travis-CI.
https://github.com/ruby/io-console/commit/de39aa6dd6 -
- Methods and their definitions can be allocated/deallocated on-the-fly.
One pathological situation is when a method is deallocated then another
one is allocated immediately after that. Address of those old/new method
entries/definitions ... -
03:07 AM Feature #14912: Introduce pattern matching syntax
- How long is the current pattern matching expected to stay experimental before becoming stabilized? (And thanks for the work so far on it!)
-
- * Default VMIN and VTIME to minimum input.
* Disable parity check bits explicitly.
* Disable all bits for flow control on input.
Co-Authored-By: NARUSE, Yui <naruse@airemix.jp>
https://github.com/ruby/io-console/commit/5ce201a686 -
- It is by design. See "Scope" section of https://github.com/ruby/ruby/blob/master/doc/syntax/modules_and_classes.rdoc
-
- What is the purpose of writing the code in three different files? It only looks to me that it is making things more complicated.
A::B.foo and A::C.foo are undefined, and return a NoMethodError, as expected. If you meant A::B#foo and A... -
- I added this to https://bugs.ruby-lang.org/issues/16393
-
- * [Feature #16419] FrozenError.new ignores receiver: (znz)
* It seems that no one have strong opinion. So I want matz to decide.
* [Feature #16420] Warning[:experimental]=false (znz)
* I heard many warnings may make users trying p...
12/17/2019
-
08:13 PM Feature #16430 (Rejected): Resultion of constants in enclosing class/module affected by how nested classes/modules are declared
- I'm not sure if this is intentional (in which case it really isn't documented anywhere, and probably should be) or a bug, but imagine the following code:
``` ruby
# lib/a.rb
module A
FOO = :BAR
end
# lib/a/b.rb
require_relat... -
05:19 PM Revision 25447724 (git): Skip optimized method check for most method IDs
- Previously every time a method was defined on a module, we would
recursively walk all subclasses to see if the module was included in a
class which the VM optimizes for (such as Integer#+).
For most method definitions we can tell immedi... -
- It's a Ruby bug if this ever happens check it as an assertion instead of
paying the cost of the check every time. -
- We know that this is a heap-allocated object (a CLASS, MODULE, or
ICLASS) so we don't need to check if it is an immediate value. This
should be very slightly faster. -
- Avoids genereating a "throwaway" sentinel class serial. There wasn't any
read harm in doing so (we're at no risk of exhaustion and there'd be no
measurable performance impact), but if feels cleaner that all class
serials actually end up ... -
- rb_clear_method_cache_by_class calls rb_class_clear_method_cache
recursively on subclasses, where it will bump the class serial and clear
some other data (callable_m_tbl, and some mjit data).
Previously this could end up taking a long t... - 03:08 PM Revision 8a40dce0 (git): * 2019-12-18 [ci skip]
-
- This reverts commit 65731a1330ed02691e3c343d2feeb8d0a72af3fd.
-
- https://github.com/ruby/io-console/commit/8ed5a2bcc3
-
- https://github.com/ruby/io-console/commit/b8411689a5
-
- https://github.com/ruby/io-console/commit/9e3ec5b936
-
-
- > I often need to check if an array have duplicate elements.
Makes sense to me; I have had situations where I needed this
too in the past (including situations for non-unique entries
in an Array), so I agree on the general use case ... -
- I propose Array#uniq?.
I often need to check if an array have duplicate elements.
This method returns true if no duplicates are found in self, otherwise returns false.
If a block is given, it will use the return value of the block... - 01:33 PM Revision ffa7cece (git): [rubygems/rubygems] Move `require uri` to source_list
- https://github.com/rubygems/rubygems/commit/c6b4df5bfc
-
-
- I want to make point for the default gem promotion.
One of Ruby's goal is to be very appealing to developers. did_you_mean is very helpful in this regard. As are well written error messages. Or source_locations of Ruby methods. Or a w... -
- One of the points made in #16363 was:
> so we can always reliably require it whenever we want to.
That is not true anymore, because did_you_mean is always required when RubyGems are enabled since [1]:
~~~
$ ruby -e 'puts $LOADE... -
- This assertion is not needed because we found the bug.
ba11a74745. -
- 古い環境で古い clang を使っている人はいなさそうなので閉じて良さそうです。
-
08:29 AM Bug #16426 (Third Party's Issue): error generating scaffold
- Hello, I am following the Oding project tutorital to create my first ruby on rails app and I am trying to generate a scaffold and I am given the following error. At the end it aborts it. Any idea of what is going wrong?
michaelswrig... -
- 07:56 AM Revision b226a681 (git): [rubygems/rubygems] require `uri` in source.rb
- https://github.com/rubygems/rubygems/commit/8de4d0272e
-
- For better readability.
-
05:21 AM Revision d616926a (git): Remove an unnecessary variable
-
05:14 AM Revision 8f52604b (git): Remove unnecessary double bangs from Pathname#root?
-
- https://github.com/ruby/logger/commit/fb3eff153f
-
-
-
- TCSAFLUSH discards the buffer read before the mode change, which makes
IRB ignore the buffer input immediately after invoked. TCSANOW
preserves the buffer.
https://github.com/ruby/io-console/commit/b362920182 -
- In raw mode with interrupt enabled.
https://github.com/ruby/io-console/commit/e9e8e3ff17 -
- https://github.com/ruby/io-console/commit/4f01db102d
-
- https://github.com/ruby/io-console/commit/a49462ed97
-
-
04:19 AM Revision a1713bed (git): [ruby/reline] Implement completion_append_character
- There is already the possibility to set Reline.completion_append_character.
However, it is not used by the line editor.
https://github.com/ruby/reline/commit/ab798931b9 -
- Including dsusp, lnext, and so on.
https://github.com/ruby/reline/commit/a44abe586b -
-
-
-
-
- Set up-to-date flag always when calling tzset().
-
- Applied in changeset commit:git|d6a2bce64a7fa1099e507e1d36b5f1533f42f60f.
----------
time.c (find_time_t): fix round-to-zero bug
`find_time_t` did not work correctly for year older than the Epoch
because it used C's integer division (w... -
- Thanks, I will commit this soon.
```diff
diff --git a/time.c b/time.c
index 25b843e405..ef8a995da4 100644
--- a/time.c
+++ b/time.c
@@ -3365,12 +3365,12 @@ find_time_t(struct tm *tptr, int utc_p, time_t *tp)
*tp = guess_... -
01:13 AM Bug #16159: rubyspec about time fails in Asia/Kuala_Lumpur timezone
- mame (Yusuke Endoh) wrote:
> By further investigation, now I think that it is a bug of Time.new.
> ...
Use DIV already defined in time.c.
The patch seems fine except that. -
- Seems a bug indeed. Using java.time (by using TruffleRuby) gives the same result as the patch.
-
- With the patch applied, `Time.new(1933)` works as expected in Asia/Kuala_Lumpur
```
$ TZ=Asia/Kuala_Lumpur ./miniruby -e 'p Time.new(1933)'
1933-01-01 00:20:00 +0720
``` -
- By further investigation, now I think that it is a bug of Time.new.
`find_time_t` in time.c uses integer division like `((tptr->tm_year-69)/4)`. Because 1933 is older than 1970, `tptr->tm_year` is negative. The result of division is... -
- `find_time_t` did not work correctly for year older than the Epoch
because it used C's integer division (which rounds negative to zero).
For example, `TIme.new(1933)` returned a wrong time whose year is 1922
in Asia/Kuala_Lumpur because... -
- I think the use case has been described and ruby users may agree
that this could be an improvement (e. g. such as the example given
by nobu for more succinct code).
#dig on Array, Hash and Struct makes sense; for Thread perhaps the
...
12/16/2019
-
11:25 PM Bug #16413 (Closed): Net::FTP#put で書き込みエラーが発生した時のコネクション切断
- Applied in changeset commit:git|5be34d6a3310065850c0c530db6936415124b5d9.
----------
ensure to close the data connection [Bug #16413] -
-
-
- Using `Time.new(year,1,2)` + a comment indicating might be a workaround, but it's not exactly nice.
-
- Isn't it weird that
```
$ TZ=Asia/Kuala_Lumpur ruby -e 'p Time.new(1933)'
1932-12-31 00:00:00 +0700
```
i.e., Time.new(1933) returns a time in 1932?
If that's actually what native calls return and not a bug then let's use `with_t... -
- There is no 00:00:00 1 Jan. 1933 in the timezone Asia/Kuala_Lumpur
```
$ TZ=Asia/Kuala_Lumpur irb
irb(main):001:0> t = Time.new(1932, 12, 31, 23, 59, 59)
=> 1932-12-31 23:59:59 +0700
irb(main):002:0> t + 1
=> 1933-01-01 00:20:00 ... -
- Hi @normal , I've added your email address to the list.
-
06:59 PM Misc #14632: [ANN] git.ruby-lang.org
- @hsbt: please add normal@ruby-lang.org
(I don't have a GH account, will not use JS or agree to their Terms of Service)
Btw, ruby-core -> redmine mail is still broken:
[ruby-core:96254] [Feature #16131] (posted recently)
original r... -
- @duerst Yes, thanks!
-
- 06:31 PM Revision b12bc3b0 (git): thread.c: avoid needless read after select|ppoll
- We do not need to issue pipe|eventfd read(2) syscall
unless select, ppoll|poll declares the FD needs reading. -
04:43 PM Misc #16393: DevelopersMeeting20191220Japan
- * [Bug #16260] Symbol#to_proc behaves like lambda, but doesn't acknowledge it (alanwu)
* This should make `Proc#lambda?` more accurate.
* [Bug #16406] `(lambda_proc << normal_proc).lambda?` should return false (alanwu)
* I think t... -
- The benchmark has been greatly improved (about 3 times) since this ticket was open. So I close this ticket.
Still, it might be worth investigating the commits in question, but unfortunately, there is no one who is willing to do. If ... -
-
- @znz これってまだ開いておく必要あります？
-
- RFC 7230, which obsoletes RFC 2616, says "Note: Some non-conformant implementations send the "deflate" compressed data without the zlib wrapper". But it doesn't say the actual fallback algorithm.
https://httpwg.org/specs/rfc7230.html#de... -
- https://zlib.net/zlib_faq.html#faq39
> 39 . What's the difference between the "gzip" and "deflate" HTTP 1.1 encodings?
> ...
> Bottom line: use the gzip format for HTTP 1.1 encoding. -
- No! RFC2616 says:
```
deflate
The "zlib" format defined in RFC 1950 [31] in combination with
the "deflate" compression mechanism described in RFC 1951 [29].
```
https://tools.ietf.org/html/rfc2616#section-3.... -
- I've changed Net::HTTP to support (raw) deflate correctly. Now it works correctly.
```
$ ./local/bin/ruby -ropen-uri -e 'URI.open("https://www.hoveround.com")'
```
@nkmrya Thank you for the great investigation. -
- Applied in changeset commit:git|5105240b1e851410020b3b3f1a2bead7ffdd4291.
----------
lib/net/http/response.rb: support raw deflate correctly
Net::HTTP had used `Zlib::Inflate.new(32 + Zlib::MAX_WBITS)` for all
content encoding (deflate... -
- I can reproduce the issue with "GDBM version 1.8.3. 10/15/2002", but not with "GDBM version 1.18.1. 27/10/2018" nor "Berkeley DB 5.3.28: (September 9, 2013)". It might be fixed on the side of GDBM. Please try a newer version.
-
- The suggested change is harmful. It will break delegation: `def foo(&blk); bar(&blk); end; foo()`.
Ruby allows redefinition of `Integer#/` which is incredibly harmful, so it is possible to allow `NilClass#to_proc`. But do you really... -
03:02 PM Bug #13144 (Closed): prettyprint doesn't detect cycles when wrapped inside Delegators
- Applied in changeset commit:git|6a75a460536a32fb62184dc5d8d66ddd737a9bad.
----------
Make prettyprint’s cycle detection aware of Delegator instances
Fixes [Bug #13144]
Co-Authored-By: Nobuyoshi Nakada <nobu@ruby-lang.org> - 03:01 PM Revision c6b1342d (git): * 2019-12-17 [ci skip]
-
- Fixes [Bug #13144]
Co-Authored-By: Nobuyoshi Nakada <nobu@ruby-lang.org> -
- This reverts commit 5105240b1e851410020b3b3f1a2bead7ffdd4291.
In RFC 2616:
```
deflate
The "zlib" format defined in RFC 1950 [31] in combination with
the "deflate" compression mechanism described in RFC 1951 [29].
``... -
- Net::HTTP had used `Zlib::Inflate.new(32 + Zlib::MAX_WBITS)` for all
content encoding (deflate, zlib, and gzip).
But the argument `32 + Zlib::MAX_WBITS` means zlib and gzip decoding
with automatic header detection, so (raw) deflate compr... -
02:01 PM Revision d6fd3903 (git): Enhancements for ENV doc
-
- Why is this needed?
It seems having no effects.
```diff
--- a/configure.ac
+++ b/configure.ac
@@ -3096,9 +3096,6 @@ AC_ARG_ENABLE(multiarch,
[multiarch=], [unset multiarch])
AS_IF([test ${multiarch+set}], [
AC_DEFI... -
- `Thread` has `#[]` method like as `Array`, `Hash`, `Struct` and so on, but no `#dig`.
For instance, `PP::PPMethods#check_inspect_key` in pp.rb can be simplified with the combination of this method and safe navigation operator.
From... -
- I'm working this at https://github.com/ruby/ruby/pull/2174
But it's still WIP status. I'm going to try in the next year. -
- Frankly speaking, it looks very expensive for Ruby to support Haiku as it is very different from the Unix-like system's convention.
If you really want to fix the issue, please submit the issue to each upstream, as nobu said. I recomm... -
-
- As I said in #15795, there is no platform maintainer for Haiku. Let's discuss the issue on that ticket.
-
- Unfortunately, there is no platform maintainer for Haiku. Could you investigate the issue yourself, and try to create a patch? If it is reasonably small, we may import it to the core. Thank you.
-
- Thank you for the reporting. It does not reproduce now. IRB is now actively developed, so I guess it is already fixed. If not, feel free to reopen.
-
10:29 AM Bug #16342: macOS + Ruby 2.6.0 以降で拡張ライブラリ内での waitpid() コールに失敗する
- 解決策を持っているわけでも何でもないのですが、気になった事項です。
Process#forkに対応するRuby本体のprocess.cを読むとC言語のレベルでfork(2)を行う前にいくつかの前準備を行っています。一方、GitHubのsamble.cを拝見すると単純にfork(2)を呼び出していますので、そういった前準備を行わないわけなので、何が起きても不思議がない気がします。
fork(2)自体はProcess#forkを使用する様に変更できれば、回避できる問題の... -
- 頻度の問題で、2.6.0 以前でもこの問題は起きえます。多少人為的ですが、irbで↓のようにやれば Interrupted system call になりました。
```
irb(main):001:0> RUBY_VERSION
=> "2.5.7"
irb(main):002:0> trap(:WINCH) { }
=> "SYSTEM_DEFAULT"
irb(main):003:0> load "sample.rb" # ロード開始直後にターミナ... -
- The equation shall hold for every call cache. However prior to this
changeset cc->me could be updated without also updating cc->def. Let's
make it sure by introducing new macro named CC_SET_ME which sets cc->me
and cc->def at once. -
-
-
- Thanks. I think this is not a bug. Closing this issue.
I guess there may be room for improvement in Net::HTTP (and Excon?), though. Investigation and a patch are welcome. -
- Applied in changeset commit:git|04e95f8985066b4ae1e97b045efda98bdefa07bd.
----------
Get rid of infinite recursion at loading transcoder
Disable encoding US-ASCII path to filesystem on Windows too.
[Bug #16392] -
- Disable encoding US-ASCII path to filesystem on Windows too.
[Bug #16392] -
- Applied in changeset commit:git|d2f04d332f2ff04f25202ed38e23de526a9aea46.
----------
Kernel#abort without arguments should print error info
[Bug #16424] -
- [Bug #16424]
-
- I cannot reproduce the issue on Ubuntu with ext4 filesystem.
```
$ make test-all TESTS="spec/ruby/core/file/utime_spec.rb" MSPECOPT="-fs"
ruby 2.7.0dev (2019-12-09T11:48:35Z master c2dc27d643) [x86_64-linux]
[| | ==================... -
- The patch has been already merged. Closing, thanks.
-
- Thank for submitting the log. Unfortunately, the crash log has almost no information to make debugging possible. Could you paste the console output too? If possible, the source code might be helpful. Also, Ruby 2.3.6 is already EOL. ...
-
- 01:05 AM Revision 2344d8ac (git): * 2019-12-16 [ci skip]
- 01:05 AM Revision f47c3824 (git): gc.h: Add SET_MACHINE_STACK_END specific for Powerpc64 (#1767)
- Currently we are not able to grab the correct end/top of the stack on
powerpc64 due to the fact that it uses the fallback function.
The fallback function does not return the correct top of the stack
because it adds a new frame and the r...
12/15/2019
-
- A very small but important correction:
mame (Yusuke Endoh) wrote:
> All the Ruby committers,
> ...
This is an important reminder: You will not be able to push any change to git.ruby-lang.org after 1st Jan.
2020 unless your committe... -
- All the Ruby committers,
This is an important reminder: You will not be able to push any change to git.ruby-lang.org after 1st Jan. 2020 unless your committer email address is not listed in https://github.com/ruby/ruby-commit-hook/blo... -
02:51 PM Bug #16392: test-all failure Windows - TestEncoding#test_nonascii_library_path
- @nobu,
In a ps window:
```
ruby 2.7.0dev (2019-12-12T23:58:42Z master 1140625cd3) [x64-mswin64_140]
Encoding filesystem Windows-1252
Encoding locale IBM437
```
In both cmd & ps windows:
```
ruby 2.7.0dev (2019-12-15T07:4... -
- What encodings are used as the "filesystem" and "locale"?
-
- I have feelings non unlike @Eregon's. Citing from my comment in [remove method-reference operator](https://bugs.ruby-lang.org/issues/16275) (which removal may, or may not be a good thing, but that's not the point currently):
> ....
>... -
- Some more thoughts on a similar subject by the author of RuboCop in a blog post:
https://metaredux.com/posts/2019/12/06/ruby-where-do-we-go-now.html
There are interesting discussions in the Reddit of that blog post:
https://www.redd... -
- Applied in changeset commit:git|e8c62836a6292bf2f691de458b24ea50c51b452a.
----------
IO#set_encoding_by_bom should err when encoding is already set
Except for ASCII-8BIT. [Bug #16422] -
- Except for ASCII-8BIT. [Bug #16422]
-
- https://github.com/ruby/ruby-commit-hook/commit/ddc83bea6fe6f4696e140b605db10ef0c922bba6
-
- `.mailmap` can be used To show canonical names and email addresses.
It is helpful for statistics of committers. -
- closed server doesn't have useful info.
So call inspect before close.
And add local debug code in comment.
https://rubyci.org/logs/rubyci.s3.amazonaws.com/unstable11x/ruby-master/log/20191215T092405Z.fail.html.gz
```
1) Failure:
IMAPT... -
- ruby_2_6 r67835 merged revision(s) ff41663403d3eb76d95f465cb94e14d2faaa04d1.
-
- Fix memory corruption in Enumerable#reverse_each [ruby-dev:50867]
[Bug #16354]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_6@67835 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
08:22 AM Feature #15829: Object#then_if(condition){}
- How about `call`ing a condition object if it's callable instead of simply using value as a condition?
```
class Object
def then_if(condition, &block)
if condition
if (condition.respond_to?(:call) && condition.call(self... -
- ruby_2_6 r67834 merged revision(s) 203b7fa1ae8cc40d41c38d684f70b3fea7fae813,0131fab749fb76800de09e49a6abf910201c5b58,9c0cd5c569ba22bc68d1a77ad6580a275cd99639,c144d7215b3444fa36d28d540b60170a5b30743d.
-
- Guard static variable first
* ext/openssl/ossl_asn1.c (Init_ossl_asn1): register the static
variable to grab an internal object, before creating the object.
otherwise the just-created object could get collect... -
-
07:41 AM Revision 9f431600 (git): Delegate should be supported now
- Since `did_you_mean` dropped its dependency on it.
-
-
-
-
- ruby_2_6 r67833 merged revision(s) 463092b84da7933f307cc8747f948f68ef19f5fd.
-
- Update rake-12.3.3.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_6@67833 b2dd03c8-39d4-4d8f-98ff-823fe69b080e -
- ruby_2_6 r67832 merged revision(s) c9423b016cfeab852bc5a829e55e0a11f80b3ab7,0b1e26398e018116180bf41cb63887f77d5d1b82,78ee2c245331e353e218b8fac9ca722a2bcd8fea.
-
- marshal.c: check instance variable count
* marshal.c (w_obj_each): ensure that no instance variable was
added while dumping other instance variables. [Bug #15968]
Hoisted out w_ivar_each
marshal.c: ... -
07:12 AM Misc #16407: Add .github/workflows/cygwin.yml
- > Could you investigate and fix it?
Sorry, I have no idea now...
If develop is hindered, you can revert it.
-
- https://docs.ruby-lang.org/ja/2.0.0/method/Kernel/m/abort.html
には「引数を省略した呼び出し時に $! が nil でなければその例外の メッセージを標準エラー出力 $stderr に出力」されるとありますが、 1.9 から出力されなくなっているようです。
```
% docker run -it --rm rubylang/all-ruby env ALL_RUBY_SINCE=ruby-1.6 ... -
- same as `rand(endless_range)`
Before:
```
$ ruby -e 'rand(..1)'
Traceback (most recent call last):
2: from -e:1:in `<main>'
1: from -e:1:in `rand'
-e:1:in `-': nil can't be coerced into Integer (TypeError)
```
After:
``... -
03:21 AM Bug #14387: Ruby 2.5 を Alpine Linux で実行すると比較的浅めで SystemStackError 例外になる
- I tried to reproduce this issue, but it was not reproduced today.
Is this issue fixed?
ruby:2.5-alpine
```
$ docker-compose build; docker-compose up
Building web
Step 1/2 : FROM ruby:2.5-alpine
---> 5ccd5d2d442a
Step 2/2 : COP... -
- anastasiastowers (Anastasia Stowers) wrote:
> Using the > and < symbols makes sense to compare Time only with the knowledge of the underlying unix time system being used.
I don’t think that knowledge necessary.
Rather, the time is c... -
12:20 AM Bug #16423 (Closed): Ruby Crash
- I was working with RubyMine, and tried to run the project, when it crashed and generated a report. The console output told me to submit the bug report
12/14/2019
-
- `IO#set_encoding_by_bom` docs [say](https://ruby-doc.org/core-2.7.0.preview3/IO.html#method-i-set_encoding_by_bom):
> If ios is not binmode or **its encoding has been set already**, an exception will be raised.
In reality this does... -
- @jeremyevans0 yeah, thanks, I see now.
-
04:36 PM Bug #16421 (Closed): public_send with empty keyword arguments (Ruby version < 2.7)
- You should use `ruby2_keywords`:
```ruby
def delegate(name, *args, &block)
public_send(name, *args, &block)
end
ruby2_keywords :delegate if respond_to?(:ruby2_keywords, true)
```
Then things will work correctly. Thi... -
- Probably it is something that I've just missed, so it is more a question than bug report.
In Ruby < 2.7, I'd implement "delegate everything" method as such:
```ruby
def delegate(name, *args, &block)
@something.public_send(name, *... -
- Although I can not say whether the specific API proposed here is useful or not (matz may just
have to decide on the API for general use), I agree with the statement of controlling/toggling
the "noise" level, based on different needs by... -
- Current master always warn pattern matching syntax.
It discourage users try them.
Another noisy warnings can stop by `Warning[:deprecated]=false`.
So I think the future may be more useful if `Warning[:experimental]=false` or `Warn... -
- I ran Actions CI and saved the vc140 build as an artifact:
ruby 2.7.0dev (2019-12-12T23:58:42Z master 1140625cd3) [x64-mswin64_140]
Using it locally, had the same error as MinGW.
```
1) Failure:
TestEncoding#test_nonascii_li... -
-
- This reverts commit 5f56a5fc9be9ea7b088795c2d3871c2352a020c2.
`FrozenError.new(mesg, nil).receiver` should not raise an
ArgumentError. - 03:15 PM Revision 52b1ba0b (git): * 2019-12-15 [ci skip]
-
-
-
- BTW, I think it would be good to include other Ruby implementations (cc @headius @enebo) in this discussion,
since I think we all want to support a stable API for ASTs longer term.
As a concrete case, `actionview_precompiler` is usin... -
- The ticket was closed unintentionally, I reopen it.
What's the way to mention an issue in a commit but not close it?
I clarified in the documentation that RubyVM::AbstractSyntaxTree is not stable API:
https://github.com/ruby/ruby/co... -
- Applied in changeset commit:git|b4b22b9278007b106fe40c0191f8dcf5e7e8c0f2.
----------
Clarify in the documentation that RubyVM::AbstractSyntaxTree is not stable API
* See [Feature #14844]. -
-
- * See [Feature #14844].
-
- FWIW, Java has such a concept too that they call "incubator modules" and basically just have the new experimental API under `jdk.incubator` (https://www.azul.com/openjdk-more-speed-less-haste/ for details).
I think we should introduce... -
- FWIW, Java has such a flag to enable experimental ("preview") features (I didn't know when writing this issue).
It's a single command-line flag called `--enable-preview`.
https://www.azul.com/openjdk-more-speed-less-haste/
They alre... -
- Whether to use a positional argument or a keyword argument was discussed in #15751. I don't have a strong feelings either way (one of the patches I added in #15751 supported keyword arguments), and I don't think @eregon or @nobu had str...
-
- Other exception classes with receiver accept recevier as keyword argument of new.
But FrozenError.new accept as positional argument.
I think keyword argument is better consistency.
```
% git grep 'receiver[:=]' ../error.c
../error... -
-
-
02:40 AM Revision e2708068 (git): Drop delegate dependency
- Original comment:
https://github.com/ruby/did_you_mean/commit/2f26c9ee770f28da0942c42cbc9e5800535b75e1 -
01:33 AM Feature #16039: Array#contains? to check if one array contains another array
- There's some similarity with #15198, to the point that I can re-use my suggestion from there:
It might make sense to use ary1.to_set.superset?(ary2). That way it makes explicit the fact that ary1 must be converted to a set. But Set#su...
12/13/2019
-
- Thanks for the patch.
Since there wasn't any recent movement, I submitted a pull request with it as requested by jeremyevans0.
I made sure to credit you in the commit description since you did all the work.
https://github.com/ruby... -
11:13 PM Bug #15882 (Closed): OpenSSL::X509::Name.parse usage
- Fixed in gem.
-
09:07 PM Revision f3240eb6 (git): NEWS: fix pattern matching example
-
07:58 PM Feature #14183: "Real" keyword argument
- > That was already legal if foo did not accept keyword arguments. :)
my full example was this
```rb
def foo(a: nil, **args)
args
end
foo(a: 1, 'a' => 2, 1 => 3, nil => 4, true => 5, Object.new => 6)
```
This is not le... -
- bughit (bug hit) wrote:
> So after this improvement, nonsense like this:
> ...
That was already legal if `foo` did not accept keyword arguments. :)
As keyword argument splats have always been just plain hashes, and not a separate ty... -
- So after this improvement, nonsense like this:
```rb
foo(a: 1, 'a' => 2, 1 => 3, nil => 4, true => 5, Object.new => 6)
```
becomes legal
and useful functionality like this destructuring example
```rb
words
.select { |pa... -
06:30 PM Feature #16244: Add a Time#before? and Time#after? method
- Using the > and < symbols makes sense to compare Time only with the knowledge of the underlying unix time system being used.
Although this is the way that we choose to store values of it in our systems, Time is special because it has ... - 03:40 PM Revision 835703c6 (git): * 2019-12-14 [ci skip]
-
- https://github.com/ruby/irb/commit/236590882c
-
-
-
-
-
-
-
-
- The two attached patches might improve the issue. The `0001-Store-MJIT-header-into-Ruby-versioned-directory-1.patch` includes the `ruby_version` in the header name, while the `0001-Store-MJIT-header-into-Ruby-versioned-directory-2.patch`...
-
-
- This is very likely created here:
https://github.com/ruby/ruby/blob/40026a408df5e3576380f6c1d8bf6c119fa2e32b/common.mk#L241
But it is not clear to me under what conditions this directory is used. -
-
-
- Now, C functions written by __builtin_cexpr!(code) and others are
named as "__builtin_inline#{n}". However, it is difficult to know
what the function is. This patch rename them into
"__builtin_foo_#{lineno}" when cexpr! is in 'foo' method. -
- > Please don't put ideological thoughts on me, that's not a good basis for debate.
Oh no, I'm not. Sorry for the misunderstanding. I was mostly replying to comments in #8948, but I ended up posting here because this is the ticket that...