Page MenuHomePhabricator
Feed Advanced Search

Jan 14 2016

epriestley added a comment to T10150: Project Tag Links work Intermitently.

I'm not really sure what's going on. The limit on the slug column is 128 characters, and that project name is nowhere near that (we should raise a better error message than we do, but I'm not sure how you're hitting the issue in the first place).

Jan 14 2016, 4:45 PM · Projects, Bug Report
epriestley added a comment to T10150: Project Tag Links work Intermitently.

Can you show me the output of bin/storage adjust? (For now, decline to apply any changes it suggests.)

Jan 14 2016, 4:42 PM · Projects, Bug Report
epriestley added a comment to T10150: Project Tag Links work Intermitently.

Prior to T9551 (circa October 12), projects could generate with invalid hashtags, but I don't think these ever worked (at least, many of them definitely did not work).

Jan 14 2016, 4:02 PM · Projects, Bug Report
epriestley added a comment to T10150: Project Tag Links work Intermitently.

The original report in Q272 is almost certainly unrelated.

Jan 14 2016, 3:55 PM · Projects, Bug Report
epriestley added a comment to T10004: ApplicationEditor Errata.

I think T10116 may have fixed the "actions ignored" issue -- does it still reproduce for you after that?

Jan 14 2016, 3:22 PM · ApplicationEditor
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 3:18 PM · Projects
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 3:06 PM · Projects
michaeloa awarded T10118: regex for parsing severity codes in ArcanistCppLinter is wrong a Like token.
Jan 14 2016, 3:03 PM · Lint, Arcanist, Bug Report
epriestley closed T10118: regex for parsing severity codes in ArcanistCppLinter is wrong as Resolved.

Resolved by D15019.

Jan 14 2016, 3:00 PM · Lint, Arcanist, Bug Report
epriestley renamed T10118: regex for parsing severity codes in ArcanistCppLinter is wrong from regex in ArcanistCppLinter is wrong to regex for parsing severity codes in ArcanistCppLinter is wrong.
Jan 14 2016, 2:59 PM · Lint, Arcanist, Bug Report
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 2:54 PM · Projects
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 2:02 PM · Projects
epriestley renamed T10148: diffusion.rawdiffquery can not encode/transport all diffs (e.g. to non-utf8 files) from Host returned HTTP/200, but invalid JSON data in response to a Conduit method call to diffusion.rawdiffquery can not encode/transport all diffs (e.g. to non-utf8 files).
Jan 14 2016, 1:30 PM · Conduit, Daemons
epriestley added a comment to T10148: diffusion.rawdiffquery can not encode/transport all diffs (e.g. to non-utf8 files).

You likely have a diff which isn't JSON-encodable (for example, it affects non-utf8 files). T5955 is probably the pathway forward here.

Jan 14 2016, 1:30 PM · Conduit, Daemons
epriestley closed T10149: Export of Phriction document as Resolved.

You can print it to a PDF.

Jan 14 2016, 1:26 PM · Phriction, Feature Request
epriestley added a comment to T10132: Unable to import a private git repo hosted on GitHub.

I wonder why path "/var/repo/M/" is empty?

Jan 14 2016, 1:26 PM · Diffusion
epriestley added Q278: Delete a diff? (Answer 307).
Jan 14 2016, 1:23 PM
joshuaspence added a comment to M1450: Projects v3.

Very nice.

Jan 14 2016, 7:48 AM · Projects
joshuaspence created T10148: diffusion.rawdiffquery can not encode/transport all diffs (e.g. to non-utf8 files).
Jan 14 2016, 7:43 AM · Conduit, Daemons
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 1:56 AM · Projects
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 14 2016, 1:04 AM · Projects

Jan 13 2016

epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 13 2016, 11:59 PM · Projects
epriestley closed T10139: Clarify wording used in the Differential diff "View Options" dropdown as Resolved by committing Restricted Diffusion Commit.
Jan 13 2016, 7:09 PM · Differential, Bug Report
epriestley added Q274: Problem at releeph (Answer 302).
Jan 13 2016, 6:50 PM
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 13 2016, 6:42 PM · Projects
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 13 2016, 5:56 PM · Projects
epriestley triaged T10145: Adding a panel to a dashboard multiple times as Wishlist priority.
Jan 13 2016, 5:33 PM · Dashboards (v2), Bug Report
epriestley renamed T10144: Document why projects will never affect visibility from Derive visibility from assigned projects to Document why projects will never affect visibility.
Jan 13 2016, 5:05 PM · Projects, Documentation, Feature Request
epriestley added a comment to T10144: Document why projects will never affect visibility.

Oops, collided midair there. I'm just going to redirect this into "write docs".

Jan 13 2016, 5:04 PM · Projects, Documentation, Feature Request
epriestley reopened T10144: Document why projects will never affect visibility as "Open".

We do not intend to ever implement this. It is intentional that adding and removing projects never affect policies.

Jan 13 2016, 5:03 PM · Projects, Documentation, Feature Request
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 13 2016, 4:37 PM · Projects
epriestley renamed T10143: Admins always see all Create Forms in Quick Create / Application Create Menus from Admins always see all Create Forms to Admins always see all Create Forms in Quick Create / Application Create Menus.
Jan 13 2016, 4:33 PM · Restricted Project, ApplicationEditor, Bug Report
epriestley added a project to T10143: Admins always see all Create Forms in Quick Create / Application Create Menus: ApplicationEditor.
Jan 13 2016, 4:33 PM · Restricted Project, ApplicationEditor, Bug Report
epriestley added a comment to T10143: Admins always see all Create Forms in Quick Create / Application Create Menus.

It is intentional that anyone who can configure the application (in this case, Maniphest) can edit all the forms for it (and can see them in order to edit them), so it's expected that you can manage and reconfigure the form in configuration UI.

Jan 13 2016, 4:33 PM · Restricted Project, ApplicationEditor, Bug Report
epriestley added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

f4d47d93fd8a56f918e4cfa72bf96d6997955a8d is not in the upstream. We do not support forked versions of Phabricator.

Jan 13 2016, 2:43 PM · Bug Report
epriestley added a comment to T10139: Clarify wording used in the Differential diff "View Options" dropdown.

Yeah, I agree your suggestions are more clear (and "Entire Diff" would be a little murkier).

Jan 13 2016, 2:24 PM · Differential, Bug Report
epriestley claimed T10139: Clarify wording used in the Differential diff "View Options" dropdown.
Jan 13 2016, 2:21 PM · Differential, Bug Report
epriestley added a revision to T10139: Clarify wording used in the Differential diff "View Options" dropdown: Unknown Object (Differential Revision).
Jan 13 2016, 2:21 PM · Differential, Bug Report
epriestley added a comment to T10120: Uncaught Error: JX.initBehavior(map): behavior(s) not registered: phabricator-object-selector.

We do not support the Bitnami image.

Jan 13 2016, 2:08 PM · Bug Report
epriestley closed T10140: How to clear Conpherence messages history? as Resolved.

There is no such option.

Jan 13 2016, 2:02 PM · Conpherence, Bug Report
epriestley added a comment to T10142: Consecutive "arc patch" attempt creating up to 3 branches before failing.

If you already have a branch named arcpatch-D69, it is intentional that we do not overwrite it. You may have made local changes there, and it would be very bad for us to destroy your work.

Jan 13 2016, 2:02 PM · Bug Report
epriestley added Q273: What software phabricator devs used to mockup interface. (Answer 300).
Jan 13 2016, 2:00 PM

Jan 12 2016

epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 12 2016, 10:20 PM · Projects
epriestley added a comment to T8258: Allow comments in JSON.

Minor tangential discussion in T5297, too.

Jan 12 2016, 9:42 PM · Restricted Project, libphutil, Arcanist
epriestley updated subscribers of T8258: Allow comments in JSON.
Jan 12 2016, 9:41 PM · Restricted Project, libphutil, Arcanist
epriestley merged task T10137: Add comments to .arclint json into T8258: Allow comments in JSON.
Jan 12 2016, 9:41 PM · Lint, Feature Request
epriestley added a revision to T10054: Define product direction for Project/Profile UIs: Unknown Object (Differential Revision).
Jan 12 2016, 7:50 PM · Projects
epriestley closed T10096: Incorrect remarkup rendering of bold text, that contains hashtag (#) as Resolved by committing Restricted Diffusion Commit.
Jan 12 2016, 6:03 PM · Restricted Project, Remarkup, Bug Report
epriestley added a revision to T10096: Incorrect remarkup rendering of bold text, that contains hashtag (#): Unknown Object (Differential Revision).
Jan 12 2016, 5:29 PM · Restricted Project, Remarkup, Bug Report
epriestley claimed T10096: Incorrect remarkup rendering of bold text, that contains hashtag (#).
Jan 12 2016, 5:27 PM · Restricted Project, Remarkup, Bug Report
epriestley added a comment to T10092: User reports Drydock not parallelizing working copy builds.

(And, obviously, all of this could be way more clear in the UI.)

Jan 12 2016, 4:05 PM · Drydock
epriestley added a comment to T10092: User reports Drydock not parallelizing working copy builds.

Sorry, this actually is expected, if the resource pool is totally empty to begin with. Here's why:

Jan 12 2016, 4:04 PM · Drydock
epriestley triaged T10131: Wiki pages are not completely print friendly as Wishlist priority.
Jan 12 2016, 3:13 PM · Feature Request, Phriction
epriestley closed T10108: Unable to query spaces via Conduit to find their PHID as Resolved.

Yeah, use phid.lookup.

Jan 12 2016, 3:13 PM · Spaces, Feature Request
epriestley closed T10130: Feed shows UNIX timestamps for custom date fields as Resolved by committing Restricted Diffusion Commit.
Jan 12 2016, 3:11 PM · Custom Fields, Bug Report
epriestley closed T10135: Current Viewer's Projects when NO projects joined as Resolved by committing Restricted Diffusion Commit.
Jan 12 2016, 3:11 PM · Projects, Bug Report
epriestley added a comment to T6703: Allow multiple copies of the same auth provider type.

See Planning.

Jan 12 2016, 3:10 PM · Auth
epriestley added a revision to T4705: Add an "Optional Display URI" for remote repositories in Diffusion: Unknown Object (Differential Revision).
Jan 12 2016, 3:06 PM · Restricted Project, Diffusion (v3)
epriestley claimed T10130: Feed shows UNIX timestamps for custom date fields.
Jan 12 2016, 2:19 PM · Custom Fields, Bug Report
epriestley claimed T10135: Current Viewer's Projects when NO projects joined.
Jan 12 2016, 2:19 PM · Projects, Bug Report
epriestley added a revision to T10130: Feed shows UNIX timestamps for custom date fields: Unknown Object (Differential Revision).
Jan 12 2016, 2:18 PM · Custom Fields, Bug Report
epriestley added a revision to T10135: Current Viewer's Projects when NO projects joined: Unknown Object (Differential Revision).
Jan 12 2016, 2:11 PM · Projects, Bug Report
epriestley closed T10134: 'Error parsing field "Reviewers"' when using '#' to address groups/projects. as Invalid.

I was able to reproduce the error message with the obvious, invalid input:

Jan 12 2016, 2:01 PM · Bug Report
epriestley closed T10121: Command died with status 255: ERROR 8: Array to string conversion as Resolved.

I'm going to close this out since I think there's no way for the upstream to move it forward for now, but file another issue if you hit a second wave of stuff or reopen this later if, e.g. all the fixes were terrible no-good garbage.

Jan 12 2016, 1:39 PM · Maniphest, Bug Report
epriestley added a comment to T10118: regex for parsing severity codes in ArcanistCppLinter is wrong.

I'll upstream regex changes if they include unit tests.

Jan 12 2016, 1:35 PM · Lint, Arcanist, Bug Report
epriestley added a comment to T10134: 'Error parsing field "Reviewers"' when using '#' to address groups/projects..

After the above, I ran arc diff origin/master --trace to update the revision. Got approximately the same screen (with "Differential Revision" now populated), saved and exited, got a clean update (see D15002).

Jan 12 2016, 1:34 PM · Bug Report
epriestley added a comment to T10134: 'Error parsing field "Reviewers"' when using '#' to address groups/projects..

Oh, you're reporting that updating causes the error, sorry. Let me try that.

Jan 12 2016, 1:33 PM · Bug Report
epriestley added a comment to T10134: 'Error parsing field "Reviewers"' when using '#' to address groups/projects..

I can't reproduce this. Here's what I did:

Jan 12 2016, 1:32 PM · Bug Report
epriestley merged task T10133: Multiple LDAP authentication providers into T6703: Allow multiple copies of the same auth provider type.
Jan 12 2016, 12:35 PM · Feature Request
epriestley updated subscribers of T6703: Allow multiple copies of the same auth provider type.
Jan 12 2016, 12:35 PM · Auth
epriestley added a comment to T10132: Unable to import a private git repo hosted on GitHub.

Upgrade Phabricator to rP5592e59 or newer, then try again.

Jan 12 2016, 12:34 PM · Diffusion
epriestley closed T10129: How to Enable Options in "Create Task" field as Resolved.

https://secure.phabricator.com/book/phabricator/article/forms/

Jan 12 2016, 12:31 PM · Feature Request
epriestley renamed T7125: Provide tooling to for periodic reminders (like renewing domains and update SSL certificates) from Provide tooling to remind administrators to renew domains and update SSL certificates to Provide tooling to for periodic reminders (like renewing domains and update SSL certificates).
Jan 12 2016, 12:31 PM · Calendar
epriestley merged task T10128: Recurring Tasks into T7125: Provide tooling to for periodic reminders (like renewing domains and update SSL certificates).
Jan 12 2016, 12:30 PM · Feature Request
epriestley updated subscribers of T7125: Provide tooling to for periodic reminders (like renewing domains and update SSL certificates).
Jan 12 2016, 12:30 PM · Calendar
mna awarded T10034: Plan the future of Maniphest task relationships a Like token.
Jan 12 2016, 9:00 AM · Maniphest
joshuaspence changed the visibility for T1941: Populating Symbol indexes fail.
Jan 12 2016, 3:55 AM · XHPAST
avivey awarded T10115: Upgrading: Unique, Sensible Repository Slugs a Mountain of Wealth token.
Jan 12 2016, 2:47 AM · Diffusion, Installing & Upgrading
epriestley created T10125: Remove Persona Login Support.
Jan 12 2016, 1:47 AM · Auth
epriestley closed T10124: Script-and-regex linter doesn't handle whole file lint correctly. as Resolved by committing Restricted Diffusion Commit.
Jan 12 2016, 1:37 AM · Bug Report
epriestley added a revision to T10124: Script-and-regex linter doesn't handle whole file lint correctly.: Unknown Object (Differential Revision).
Jan 12 2016, 1:28 AM · Bug Report
epriestley added a comment to T10124: Script-and-regex linter doesn't handle whole file lint correctly..

D15000 will make capturing no text mean "the whole file", and documents the behavior. You can either change your regexp to not capture "0":

Jan 12 2016, 1:26 AM · Bug Report
epriestley added a revision to T10124: Script-and-regex linter doesn't handle whole file lint correctly.: Unknown Object (Differential Revision).
Jan 12 2016, 1:24 AM · Bug Report
epriestley added a comment to T10124: Script-and-regex linter doesn't handle whole file lint correctly..

Well, maybe that's overkill.

Jan 12 2016, 1:11 AM · Bug Report
epriestley updated subscribers of T10124: Script-and-regex linter doesn't handle whole file lint correctly..
Jan 12 2016, 1:09 AM · Bug Report
epriestley merged task T6303: Missing lint warnings where line number is unknown into T10124: Script-and-regex linter doesn't handle whole file lint correctly..
Jan 12 2016, 1:09 AM · Arcanist
epriestley added a comment to T10124: Script-and-regex linter doesn't handle whole file lint correctly..

I believe D14307 changed the behavior from "fatal horribly" to "work". It's possible that capturing 0 meant "whole file" at some point (although I'm confused by T6303 if it did) but I think that was accidental and don't believe we suggested/recommended/documented that anywhere.

Jan 12 2016, 1:09 AM · Bug Report
epriestley added a comment to T10124: Script-and-regex linter doesn't handle whole file lint correctly..

Instead of the line being 0 (meaning whole file)

Jan 12 2016, 12:44 AM · Bug Report

Jan 11 2016

epriestley added a comment to T10123: Allow short "result" message for CI builds.

See T5920#131011 for prior discussion.

Jan 11 2016, 10:27 PM · Restricted Project, Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T10122: ctrl+enter to commit in all fields.

I largely can't reproduce this. Here's what I did:

Jan 11 2016, 6:41 PM · Restricted Project, ApplicationEditor, Feature Request
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Alright, I think you should get different behavior at HEAD for both of those issues. No promises it will be correct, but let me know if those changes just uncover a second series of errors.

Jan 11 2016, 6:33 PM · Maniphest, Bug Report
epriestley added a revision to T10121: Command died with status 255: ERROR 8: Array to string conversion: Unknown Object (Differential Revision).
Jan 11 2016, 6:31 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

I also don't think it's wholly unreasonable to change the prefix rule to be stricter, but I think getting rid of the username@ stuff will moot it in all realistic cases. We could look at the rules after the next Conpherence update if we want to reintroduce the username@ rule.

Jan 11 2016, 6:27 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The mail looks like this:

Jan 11 2016, 6:23 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

Phabricator strips the prefix if it is present, but does not require it to be present, and attempts to route all mail for the domain it is given.

Jan 11 2016, 6:16 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

I intend to remove username@domain.com causing interactions with Conpherence.

Jan 11 2016, 6:12 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

(Allowing Phabricator to receive mail at domain.com and also having real mailing lists and users at that domain is still a bad idea, because the two namespaces will collide, but few normal addresses resemble Phabricator object addresses so the practical badness is low -- almost no one has D832@mycompany.com as a real address.)

Jan 11 2016, 6:11 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

The second (or third or fourth or whatever, depending on how we're counting) Conpherence issue I'm unsure about.

Jan 11 2016, 6:09 PM · Maniphest, Bug Report
epriestley added a comment to T10121: Command died with status 255: ERROR 8: Array to string conversion.

"Array to string conversion" is a bug. Once that's fixed (by D14996), the user will start getting some kind of error email instead (the bad conversion was occurring while trying to compose an email to notify the user about an error). That error email may describe an upstream bug, or may describe user error. The stack trace doesn't have enough information for me to guess.

Jan 11 2016, 6:03 PM · Maniphest, Bug Report
epriestley added a revision to T10121: Command died with status 255: ERROR 8: Array to string conversion: Unknown Object (Differential Revision).
Jan 11 2016, 6:01 PM · Maniphest, Bug Report