@bdrewery Thanks for reporting
@adamw Thanks for your help

Upload patch, so review matches what we discussed.

@adamw

I get that same output. And to confirm, that's what's currently in the tree, correct?

@bdrewery @adamw Well, so for me (from Germany in various locations/ISPs) buildings works with the patch I committed and would fail without it (just tried on a new server, since the package wasn't available).

As I will be AFK now, I quickly committed the change myself (I bumped the revision, which is not strictly necessary, but I would find it confusing otherwise + building these ports is really cheap)

In D53566#1222841, @ziaee wrote:

In D53566#1222840, @grembo wrote:

In D53566#1222838, @ziaee wrote:

Picture worth a thousand words?

I'm using 80x25 here, which I know people loathe, but 80x60 or whatever in GUI windows is I think a very common use case, or split screen that still works out to ~80.

Also I think it's more useful for quick scanning to put what its about first, like we do with commit messages.

Sorry, the picture is not visible (it says restricted file attached). You might need to modify its permissions.

Thanks, did that fix it?

I also use 80x25 most of the time, but the comment field is defined as being up to 70 chars, so there is no reason to try to strip down all port comments to fit strlen(package name + version + comment).

What about simply using pkg search arcanist | less -S?

That said, why not make it CLI while being there?

• arcanist: Phabricator CLI
• arcanist-lib: Phabricator Arcanist libraries
• phorgeit-arcanist: Phorge.it CLI
• phorgeit-arcanist-lib: Phorge.it Arcanist libraries

Well, that's totally fine with me, but I didn't do that because now we're removing syntactically significant search terms. I only removed conjunctions so far.

In D53566#1222838, @ziaee wrote:

Picture worth a thousand words?

I'm using 80x25 here, which I know people loathe, but 80x60 or whatever in GUI windows is I think a very common use case, or split screen that still works out to ~80.

Also I think it's more useful for quick scanning to put what its about first, like we do with commit messages.

While I'm not fundamentally opposed to the stylistic change, I don't understand the reasoning.