On Fri, 17 Apr 2020 18:11:08 -0700 Andrew Hewus Fresh <[email protected]> wrote: > I like this better too, but unfortunately it doesn't work. So we need > to do something else.
What do you mean with "doesn't work"? If you're refering to doing `portgen ruby activesupport` ... The check whether the gem is already ported happens currently at the end of get_deps(). Therefore dependencies are checked, but the top-level isn't. To also check the top-level requires more refactoring than just moving this part to name_new_port(). Running `portgen ruby activesupport` once will create the port in mystuff. Running again it will notice that the directory exists and say something like "Not porting ruby/ruby-activesupport, already exists in /usr/ports/mystuff" (or "Not porting devel/ruby-activesupport, already exists in /usr/ports/mystuff" if you move the logic to name_new_port()), but this is only a result of creating the directory in the first run.
