Page MenuHomePhabricator
Feed Advanced Search

Aug 2 2016

chad added a comment to T11416: fixed-width-text remarkup blocks lost max-width.

So regular comments on the feed page, nothing to do with differential or inline comments?

Aug 2 2016, 5:16 PM · PHUI, Bug Report
avivey added a comment to T11416: fixed-width-text remarkup blocks lost max-width.

Regular comments, not inline comments (Although I haven't checked those yet).

Aug 2 2016, 5:14 PM · PHUI, Bug Report
chad added a comment to T11416: fixed-width-text remarkup blocks lost max-width.

So feed and differential inline comments?

Aug 2 2016, 5:14 PM · PHUI, Bug Report
CodeMouse92 added a comment to T9640: Make Phabricator compatible with PHP7.

@michaeljs1990, is was actually not possible to install PHP5 on Ubuntu 16.04 through a safe means (an apt repository or similar) for some time.

Aug 2 2016, 5:01 PM · Prioritized, Restricted Project, Infrastructure
avivey added a comment to T11416: fixed-width-text remarkup blocks lost max-width.

Ha! It's bad in the feed page.

Aug 2 2016, 5:00 PM · PHUI, Bug Report
avivey added a comment to T11416: fixed-width-text remarkup blocks lost max-width.
Usage Exception: Linter 'versions' specifies invalid type 'libversion'. Available linters are: csharp, chmod, composer, filename, xhpast, csslint, gjslint, coffeelint, cppcheck, cpplint, flake8, golint, hlint, jshint, jsonlint, jscs, lessc, pep8, php, phpcs, puppet-lint, pyflakes, pylint, rubocop, ruby, generated, json, merge-conflict, nolint, phutil-library, script-and-regex, spelling, text, xml, checkstyle.
Aug 2 2016, 4:59 PM · PHUI, Bug Report
avivey created T11416: fixed-width-text remarkup blocks lost max-width.
Aug 2 2016, 4:58 PM · PHUI, Bug Report
chad added a project to T11414: Clicking on mock inline comment preview doesn't display the correct image: Pholio.
Aug 2 2016, 4:50 PM · Restricted Project, Pholio, Bug Report
chad edited projects for T11411: Commits to github branch close Differential revision, added: Diffusion, Documentation; removed Bug Report.
Aug 2 2016, 4:50 PM · Documentation, Diffusion
chad added projects to T11407: Make is possible to react to commits being parsed: Diffusion, Herald.
Aug 2 2016, 4:49 PM · Herald, Diffusion, Feature Request
chad added a project to T11402: Garbage collect and/or compress/archive harbormaster build unit messages: Harbormaster.
Aug 2 2016, 4:49 PM · Restricted Project, Harbormaster, Feature Request
chad added a project to T11403: Phabricator hosted git repositories appear to error with 'failed to push some refs' without any error detail..: Diffusion.

Sounds like this is resolved? Or should we attempt to catch this case?

Aug 2 2016, 4:49 PM · Diffusion, Bug Report
chad added a project to T11415: Would like people who can't edit Harbormaster plans to be able to run them: Harbormaster.
Aug 2 2016, 4:47 PM · Restricted Project, Harbormaster, Feature Request
djehuti added a comment to T11415: Would like people who can't edit Harbormaster plans to be able to run them.

Or are you just concerned that they may make mistakes

Aug 2 2016, 4:39 PM · Restricted Project, Harbormaster, Feature Request
fabe added a comment to T5055: Distribution mechanism for arc extensions.

The whole "identify the packager problem" with composer is indeed a big one and i'm not suggesting you use composer in any way for the arcanist/phabricator packages! Generally i agree that i want to primarily trust the packager and not the developer. This is the way this is handled with os packages (rpm/deb) and companies are used to the workflow: Getting new packages from a somewhat trusted/signed source (e.g. redhat/canonical), testing them yourself, signing them with your own key as well and then distributing to the internal repositories and "end users" only trust your own internal key.

Aug 2 2016, 4:18 PM · Arcanist
epriestley added a comment to T11415: Would like people who can't edit Harbormaster plans to be able to run them.

(You can't write "Run Build Plans: ..." personal rules, but you can write them in object rules, which isn't too much different.)

Aug 2 2016, 4:02 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T11415: Would like people who can't edit Harbormaster plans to be able to run them.

It's also probably bad/inconsistent that you can run builds via Herald which you could not run via the web UI.

Aug 2 2016, 4:00 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a comment to T11415: Would like people who can't edit Harbormaster plans to be able to run them.

I have mixed feelings about relaxing this. I think there are basically two parts to this:

Aug 2 2016, 3:57 PM · Restricted Project, Harbormaster, Feature Request
djehuti created T11415: Would like people who can't edit Harbormaster plans to be able to run them.
Aug 2 2016, 3:36 PM · Restricted Project, Harbormaster, Feature Request
epriestley added a revision to T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess: Unknown Object (Differential Revision).
Aug 2 2016, 3:32 PM · Documentation, Herald
epriestley added a comment to T5055: Distribution mechanism for arc extensions.

I generally expect all signing to be external, at least for the foreseeable future.

Aug 2 2016, 3:22 PM · Arcanist
fabe added a comment to T5055: Distribution mechanism for arc extensions.

It might be beneficial to generally support gpg signed commits / tags in phabricators git repos and then use the same mechanism for the arcanist packages.
Since github recently started pushing this feature a bit (https://github.com/blog/2144-gpg-signature-verification) quite a few library maintainers started signing their release tags.
And i just recently spoke to a lib maintainer about extending composer to verify all packages (on install or upgrade) against a list of authors trusted by the user (or an enterprise wide list).

Aug 2 2016, 1:46 PM · Arcanist
enckse added a comment to T9640: Make Phabricator compatible with PHP7.

It seems likely, as well, until php 7.1 is officially supported that you probably don't want to sign-up for some potential pain points by choosing to roll the dice.

Aug 2 2016, 12:35 PM · Prioritized, Restricted Project, Infrastructure
michaeloa added a comment to T11409: harbormaster.buildquery fatals on short array syntax in older PHP.

Thanks. Appreciate the quick fix on this.

Aug 2 2016, 12:18 PM · Lint, Harbormaster, Bug Report
mcorteel created T11414: Clicking on mock inline comment preview doesn't display the correct image.
Aug 2 2016, 11:55 AM · Restricted Project, Pholio, Bug Report
fschloegl awarded T5474: Support workboard column triggers which activate when a task is dropped into a column a Like token.
Aug 2 2016, 11:37 AM · Workboards (v3), Restricted Project
michaeljs1990 added a comment to T9640: Make Phabricator compatible with PHP7.

I think you are using the word "only" wrong @PascalReintjens, @CodeMouse92. What you mean is by default. You can easily install any version of PHP you want this should not be a blocker for anyone. It's a slight annoyance but so is trying to keep track of the release cycle for RHEL, Ubuntu, CentOS, Debian, OSX, ArchLinux, Gentoo, and every other distribution you can think of.

Aug 2 2016, 5:44 AM · Prioritized, Restricted Project, Infrastructure

Aug 1 2016

epriestley added a comment to T9640: Make Phabricator compatible with PHP7.

In the case of Debian 9, it looks like it won't even freeze for 6 months. If it ships with PHP 7.1, it may well run Phabricator. It is impossible for us to answer compatibility questions about hypothetical software slated for release the better part of a year from now.

Aug 1 2016, 11:53 PM · Prioritized, Restricted Project, Infrastructure
CodeMouse92 added a comment to T9640: Make Phabricator compatible with PHP7.

@PascalReintjens, correct, in the same way that it won't run under Ubuntu 16.04 (which also comes with PHP7 only).

Aug 1 2016, 11:46 PM · Prioritized, Restricted Project, Infrastructure
PascalReintjens added a comment to T9640: Make Phabricator compatible with PHP7.

Debian Stretch will come with php7 only. Am I right assuming that Phabricator won't run under Debian 9?

Aug 1 2016, 11:36 PM · Prioritized, Restricted Project, Infrastructure
Fordy added a comment to T11403: Phabricator hosted git repositories appear to error with 'failed to push some refs' without any error detail...

Thanks for your help / advice Evan!

Aug 1 2016, 11:28 PM · Diffusion, Bug Report
avivey updated the answer details for Q456: Why do phabricator hosted git repositories appear to error with 'failed to push some refs' without any error detail, even though the push works? (Answer 426).
Aug 1 2016, 11:24 PM
Fordy added Q456: Why do phabricator hosted git repositories appear to error with 'failed to push some refs' without any error detail, even though the push works? (Answer 426).
Aug 1 2016, 11:22 PM
chad updated images of M1466: Phriction v3.
Aug 1 2016, 10:22 PM
chad added a comment to M1466: Phriction v3.

Hah, I actually had the page title instead of table of contents but it felt really repetitive

Aug 1 2016, 9:21 PM
epriestley added a comment to M1466: Phriction v3.

In this particular mock it's slightly confusing to me that the left nav looks like it has a "Table of Contents" header but is showing a document hierarchy (i.e., element that looks like a header does not seem to be labeling the content beneath it). Not sure if that's a mock thing / interaction thing / whatever.

Aug 1 2016, 9:19 PM
epriestley added a comment to T11413: Support renaming Phacility instances.

It would also be nice to recognize that "oldname" became "newname" so we could redirect, continue delivering mail, etc. This is a little more involved.

Aug 1 2016, 9:03 PM · Phacility
epriestley created T11413: Support renaming Phacility instances.
Aug 1 2016, 9:01 PM · Phacility
chad created M1466: Phriction v3.
Aug 1 2016, 8:30 PM
chad added a comment to T11412: Ability to edit blog name.

Specifically if you feel this is a Phabricator issue, please provide steps we can follow on a clean/new install on http://phacility.com/. Test instances there are free to spin up any time.

Aug 1 2016, 6:39 PM · Phame, Bug Report (Needs Information)
chad closed T11412: Ability to edit blog name as Invalid.

Please see Contributing Bug Reports for how to file a proper bug report. See Support Resources for what resources are available for getting help configuring or using Phabricator.

Aug 1 2016, 6:34 PM · Phame, Bug Report (Needs Information)
epriestley added a revision to T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess: Unknown Object (Differential Revision).
Aug 1 2016, 6:31 PM · Documentation, Herald
epriestley placed T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules up for grabs.
Aug 1 2016, 6:30 PM · Applications, Herald
epriestley removed a subtask for T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess: T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.
Aug 1 2016, 6:30 PM · Documentation, Herald
epriestley removed a parent task for T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules: T9719: Herald test console doesn't appear to work for commit hooks (pre-receive), and also has a big `instanceof` mess.
Aug 1 2016, 6:30 PM · Applications, Herald
epriestley added a comment to T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.

Oh, I think I made up point (1) above -- that's already the behavior.

Aug 1 2016, 6:30 PM · Applications, Herald
epriestley claimed T7961: In Herald, don't offer Fields and Actions from uninstalled applications to users writing rules.
Aug 1 2016, 6:26 PM · Applications, Herald
Taskle added a comment to T11412: Ability to edit blog name.

I am an admin w/full permissions, but even so I've tried just doing this on my own personal blog that I created myself and can't even do that.

Aug 1 2016, 6:24 PM · Phame, Bug Report (Needs Information)
chad added a comment to T11412: Ability to edit blog name.

Specifically, if you don't have permissions to edit a blog, you should get a permissions dialog like below. Sounds like someone has been messing with the custom forms.

Aug 1 2016, 6:12 PM · Phame, Bug Report (Needs Information)
chad edited projects for T11412: Ability to edit blog name, added: Bug Report (Needs Information), Phame; removed Feature Request.
Aug 1 2016, 6:04 PM · Phame, Bug Report (Needs Information)
chad added a comment to T11412: Ability to edit blog name.

I can't reproduce this. Can you open a bug report with full reproduction steps?

Aug 1 2016, 6:04 PM · Phame, Bug Report (Needs Information)
venky added a comment to T11411: Commits to github branch close Differential revision.

Maybe link it in the "Explain why" dialog?

Aug 1 2016, 6:04 PM · Documentation, Diffusion
venky added a comment to T11411: Commits to github branch close Differential revision.

Ah, sweet. Thank you!

Aug 1 2016, 6:02 PM · Documentation, Diffusion
avivey added a comment to T11411: Commits to github branch close Differential revision.

This question has been coming up rather frequently lately; maybe we should expose the branch configuration more prominently somehow?

Aug 1 2016, 5:54 PM · Documentation, Diffusion
epriestley closed T11409: harbormaster.buildquery fatals on short array syntax in older PHP as Resolved by committing Restricted Diffusion Commit.
Aug 1 2016, 5:45 PM · Lint, Harbormaster, Bug Report
Taskle created T11412: Ability to edit blog name.
Aug 1 2016, 5:18 PM · Phame, Bug Report (Needs Information)
venky created T11411: Commits to github branch close Differential revision.
Aug 1 2016, 5:09 PM · Documentation, Diffusion
chad added a comment to T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().

Yeah, we don't support or recommend third-party extensions anywhere in the documentation. We're years away from that.

Aug 1 2016, 2:11 PM · Bug Report
mibesis added a comment to T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().

@chad My bad, let me explain.

Aug 1 2016, 1:58 PM · Bug Report
chad added a comment to T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().

No, it's not our product. It's a third party extension. Please contact that maintainer for help.

Aug 1 2016, 1:47 PM · Bug Report
mibesis added a comment to T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().

@chad Agree, but it is "in yours product/project". Does this does not count?

Aug 1 2016, 1:46 PM · Bug Report
chad closed T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns() as Invalid.

This isn't our code:

Aug 1 2016, 1:44 PM · Bug Report
mibesis updated the task description for T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().
Aug 1 2016, 1:41 PM · Bug Report
mibesis created T11410: UNRECOVERABLE FATAL ERROR -- PhabricatorProjectColumnPositionQuery::withColumns().
Aug 1 2016, 1:40 PM · Bug Report
epriestley added a revision to T11409: harbormaster.buildquery fatals on short array syntax in older PHP: Unknown Object (Differential Revision).
Aug 1 2016, 12:08 PM · Lint, Harbormaster, Bug Report
epriestley renamed T11409: harbormaster.buildquery fatals on short array syntax in older PHP from Internal server error in Arcanist to harbormaster.buildquery fatals on short array syntax in older PHP.
Aug 1 2016, 11:26 AM · Lint, Harbormaster, Bug Report
epriestley added a revision to T11409: harbormaster.buildquery fatals on short array syntax in older PHP: Restricted Differential Revision.
Aug 1 2016, 11:26 AM · Lint, Harbormaster, Bug Report
epriestley claimed T11409: harbormaster.buildquery fatals on short array syntax in older PHP.
Aug 1 2016, 11:13 AM · Lint, Harbormaster, Bug Report
michaeloa created T11409: harbormaster.buildquery fatals on short array syntax in older PHP.
Aug 1 2016, 11:11 AM · Lint, Harbormaster, Bug Report
epriestley merged T11408: Invitees for recurring event are listed in reverse order into T11326: Improve Calendar v1 Design/UI/UX.
Aug 1 2016, 10:17 AM · Design, Calendar
epriestley merged task T11408: Invitees for recurring event are listed in reverse order into T11326: Improve Calendar v1 Design/UI/UX.
Aug 1 2016, 10:17 AM · Calendar, Bug Report
ivo created T11408: Invitees for recurring event are listed in reverse order.
Aug 1 2016, 10:15 AM · Calendar, Bug Report
MZMcBride added a comment to T6502: Policy to define who can move cards in a workboard.

We have no precedents of abuse in Phabricator so far, but looking at our history with Bugzilla I will not be surprised the morning we find a first precedent.

^ I am incredibly jealous of your perspective and am really, really surprised you haven't had this problem yet!

Aug 1 2016, 9:24 AM · Projects, Wikimedia, Policy, Workboards
venky added a comment to T11397: Overcommit -> arc lint.
Aug 1 2016, 8:00 AM · Feature Request
dereckson updated the task description for T5462: How do I publish Phabricator events into remote systems?.
Aug 1 2016, 12:16 AM · Guides, Doorkeeper, Feed, Herald

Jul 31 2016

epriestley added a comment to T11407: Make is possible to react to commits being parsed.

I'd like to make the Diffusion UI more field-focused (T9713). which is maybe adjacent, although not directly related.

Jul 31 2016, 10:14 PM · Herald, Diffusion, Feature Request
yelirekim created T11407: Make is possible to react to commits being parsed.
Jul 31 2016, 9:53 PM · Herald, Diffusion, Feature Request
epriestley added a comment to T11405: Need to watch the project programmatically.

Also, please note this:

Jul 31 2016, 9:48 PM · Feature Request (Needs Information), Conduit
epriestley added a comment to T11405: Need to watch the project programmatically.

Example 1: Batch edit of more projects at once - obviously easier to run script than having to click it manually.

Jul 31 2016, 9:45 PM · Feature Request (Needs Information), Conduit
Danny_B added a comment to T11405: Need to watch the project programmatically.

How about this: we'll play the "5 whys" game described in Describing Root Problems to get to bottom of this problem. After we figure it out, you tell me why you didn't provide that information in the first place and how to improve the documentation so that it's clear what we're after. Sound good?

Fair enough. I'm totally willing to sprinkle my head with ashes after I'll find out what exactly is causing the misunderstanding of what "root cause" is. ;-) But not knowing it is obviously not helpful and leads to the same situation again.

Jul 31 2016, 9:38 PM · Feature Request (Needs Information), Conduit
Kwisatz added a comment to T3179: Importing data from external systems (like GitHub, JIRA, Trac, Asana, Trello, etc.).

In case anyone is interested, we just released a first version of "remaim", a tool for migrating Redmine issues to Maniphest tasks. Pull Requests are very welcome! https://github.com/tentwentyfour/remaim

Jul 31 2016, 9:29 PM · Guides, Doorkeeper
yelirekim added a comment to T11406: arc diff now fails at 'differential.query'.

Yeah I have a diff incoming, looks like it's working.

Jul 31 2016, 8:15 PM · Bug Report
epriestley added a comment to T11406: arc diff now fails at 'differential.query'.

Try now?

Jul 31 2016, 8:09 PM · Bug Report
epriestley closed T11406: arc diff now fails at 'differential.query' as Resolved by committing Restricted Diffusion Commit.
Jul 31 2016, 8:08 PM · Bug Report
epriestley claimed T11406: arc diff now fails at 'differential.query'.
Jul 31 2016, 8:05 PM · Bug Report
yelirekim renamed T11406: arc diff now fails at 'differential.query' from arc diff is broken to arc diff now fails at 'differential.query'.
Jul 31 2016, 7:49 PM · Bug Report
yelirekim created T11406: arc diff now fails at 'differential.query'.
Jul 31 2016, 7:23 PM · Bug Report
epriestley renamed T11401: Add ability to quote, edit, view raw, view history, and remove inline comments from Add ability to quote inline comments to Add ability to quote, edit, view raw, view history, and remove inline comments.
Jul 31 2016, 7:09 PM · Inline Comments, Restricted Project, Differential, Feature Request
epriestley added a comment to T6273: Inline comments do not support editing (Pholio, Differential, Diffusion).

See also T11401, although this started life as a Pholio task and that's now a code-specific UI task so I don't know if they're actually the same.

Jul 31 2016, 7:08 PM · Audit, Restricted Project, Diffusion, Differential, Transactions, Pholio
epriestley added a comment to T11401: Add ability to quote, edit, view raw, view history, and remove inline comments.

Yeah, I think this is largely an issue of how much space we have in the UI.

Jul 31 2016, 7:06 PM · Inline Comments, Restricted Project, Differential, Feature Request
epriestley closed T11404: Improve the performance of custom field queries as Resolved.

I deployed this stuff here. I neglected to get a production timing beforehand so I can't really claim it's actually faster, but it seems fast-ish now. Let me know if things look better and/or I ruined everything when you get around to deploying it.

Jul 31 2016, 6:48 PM · Performance, Feature Request
epriestley reopened T11405: Need to watch the project programmatically as "Open".

How about this: we'll play the "5 whys" game described in Describing Root Problems to get to bottom of this problem. After we figure it out, you tell me why you didn't provide that information in the first place and how to improve the documentation so that it's clear what we're after. Sound good?

Jul 31 2016, 6:29 PM · Feature Request (Needs Information), Conduit
epriestley added a comment to T11404: Improve the performance of custom field queries.

I don't immediately have an attack on the JIRA thing (it should be less expensive in normal cases), and don't want to do the Auth stuff without more planning, and nothing else leaps out at me as having a similar sort of value for the cost.

Jul 31 2016, 5:08 PM · Performance, Feature Request
epriestley added a revision to T11404: Improve the performance of custom field queries: Unknown Object (Differential Revision).
Jul 31 2016, 4:49 PM · Performance, Feature Request
epriestley added a revision to T11404: Improve the performance of custom field queries: Unknown Object (Differential Revision).
Jul 31 2016, 4:33 PM · Performance, Feature Request
epriestley added a comment to T11404: Improve the performance of custom field queries.

There's still quite a bit of room here. Locally, I see a 220ms call for differential.revision.search. Here are savings which look fairly easy to capture:

Jul 31 2016, 4:25 PM · Performance, Feature Request
epriestley added a revision to T11404: Improve the performance of custom field queries: Unknown Object (Differential Revision).
Jul 31 2016, 4:06 PM · Performance, Feature Request
epriestley added a revision to T11404: Improve the performance of custom field queries: Unknown Object (Differential Revision).
Jul 31 2016, 3:46 PM · Performance, Feature Request
epriestley added a revision to T11404: Improve the performance of custom field queries: Unknown Object (Differential Revision).
Jul 31 2016, 3:29 PM · Performance, Feature Request
epriestley added a comment to T11404: Improve the performance of custom field queries.

That sort of leaves you in trouble with custom edge-based fields like the one above, but a reasonable short term approach is the one you've already taken:

Jul 31 2016, 3:08 PM · Performance, Feature Request