Honestly, I'd be more than happy if the .gemspec files of default gems in repository used similar position and format. For example compare these:

https://github.com/ruby/ruby/blob/master/lib/prime.gemspec
https://github.com/ruby/ruby/blob/master/lib/irb/irb.gemspec

These files are for no reason placed in different subdirectory. Would it be problem, if I submitted patch shuffling these around? Of course the best location would be the source root directory. That way, the file list would be usable without too much magic. Second and third options are placing all the .gemspec files either directly into the "lib" directory or have the subdirectories for each gem. Both are possible and both should be easily doable.

Also, it is pain that the executables are placed in "libexec" directory, but the .gemspec files does not respect the placement. They sometimes specify "bin", other times "exe". Sometimes, the location is even modified comparing to the upstream, such as in Bundler case [1]. Frankly, I consider this case the worst of all, because it cannot be synchronized with upstream and places yet another exception.

@voxik Can you rebase with the current master branch?

@voxik Can you rebase with the current master branch?

I'll take a look. It seems there are now 55bf0ef and 8dab71b apart from my changes. Have to investigate their impact

@voxik Can you rebase with the current master branch?

Done.

These files are for no reason placed in different subdirectory. Would it be problem, if I submitted patch shuffling these around? Of course the best location would be the source root directory. That way, the file list would be usable without too much magic. Second and third options are placing all the .gemspec files either directly into the "lib" directory or have the subdirectories for each gem. Both are possible and both should be easily doable.

Source root directory sounds like the best place indeed and would allow much easier gemspec syncronization with upstream.

@voxik Fixes in #2930 were merged, so you need to rebase here.

@voxik Ping, I'd really like this PR to go in!

Sorry, it is still on my todo list, but I am buried under other stuff ATM :/

@voxik I plan to work on this at some point too, so if I end up beating you to it, I'll let you know so we don't duplicate work 👍.

@deivid-rodriguez sure thing 👍

@voxik This is just a friendly reminder for you to rebase. Thank you.

Rebase had removed the commits, and closed unintentionally.

Thx for merging 🔝

I think there was omitted one bit. I was using this hunk for some purpose:

@@ -788,6 +794,11 @@ class Gem::Installer
     end
   end
 
+  default_spec_file = instance_method(:default_spec_file)
+  define_method(:default_spec_file) do
+    with_destdir(default_spec_file.bind(self).call)
+  end
+
   generate_bin_script = instance_method(:generate_bin_script)
   define_method(:generate_bin_script) do |filename, bindir|
     generate_bin_script.bind(self).call(filename, bindir)

Of course I don't remember anymore without more investigation 🤷‍♂️

Tests failed with overriding default_spec_file.

def default_spec_file
  with_destdir(super)
end

Ah, ok. Thx. I'll try to get back to this once the situation in Fedora stabilized and I have some free cycles.