Page MenuHomePhabricator
Feed Advanced Search

May 17 2016

krisbuist added a comment to T10645: Accepting a closed diff causes diff to be re-opened.

Oh, my apology for not replying on your previous comment, somehow the notification slipped through and I'm not a very regular visitor of this Phabricator instance.

May 17 2016, 1:12 PM · Differential, Bug Report
epriestley merged task T10645: Accepting a closed diff causes diff to be re-opened into T10967: Move Differential reviewers back to a dedicated database storage table.
May 17 2016, 12:58 PM · Differential, Bug Report
epriestley added a comment to T10645: Accepting a closed diff causes diff to be re-opened.

I'm going to merge this into T10967, since after digging around in the code a bit recently it seems unlikely that we'll pursue the infrastructure changes required to fix this outside of the context of that larger infrastructure change.

May 17 2016, 12:58 PM · Differential, Bug Report
epriestley added a revision to T10975: Unable to edit applications in OAuth Server - edge table missing: Restricted Differential Revision.
May 17 2016, 11:54 AM · OAuthServer, Bug Report
epriestley added a comment to T10975: Unable to edit applications in OAuth Server - edge table missing.

I can only reproduce this if (3) is changing "Visible To", exactly: i.e., same file policy scrambling issue as T10778. Should have the same fix.

May 17 2016, 11:48 AM · OAuthServer, Bug Report
eadler added a comment to T10975: Unable to edit applications in OAuth Server - edge table missing.

Ref T10778

May 17 2016, 10:42 AM · OAuthServer, Bug Report
bcooksley created T10975: Unable to edit applications in OAuth Server - edge table missing.
May 17 2016, 7:58 AM · OAuthServer, Bug Report
epriestley added a comment to T10974: building dependency cycle hangs with no error.

Likely the same as T6815 since I think they're the same code.

May 17 2016, 12:52 AM · Differential, Bug Report
epriestley merged task T10974: building dependency cycle hangs with no error into T6815: Creating a cyclic task dependency in Maniphest no longer produces a nice error dialog.
May 17 2016, 12:52 AM · Differential, Bug Report
avivey created T10974: building dependency cycle hangs with no error.
May 17 2016, 12:27 AM · Differential, Bug Report

May 16 2016

epriestley closed T10972: changing repo keys doesn't update "Used by" on them as Resolved by committing Restricted Diffusion Commit.
May 16 2016, 11:39 PM · Bug Report
epriestley added a revision to T10972: changing repo keys doesn't update "Used by" on them: Restricted Differential Revision.
May 16 2016, 11:38 PM · Bug Report
allixsenos updated the task description for T10972: changing repo keys doesn't update "Used by" on them.
May 16 2016, 9:46 PM · Bug Report
allixsenos created T10972: changing repo keys doesn't update "Used by" on them.
May 16 2016, 9:46 PM · Bug Report
michaeljs1990 added a comment to T10969: Incorrect Phriction link in email.

Not sure if this should be another bug report but the / that is added at the end causes issue when using anchor tags i.e if [[ ../2016.19#gotolocation ]] was the markup the link would also be broken by adding a / at the end.

May 16 2016, 5:08 PM · Phriction, Bug Report
richardvanvelzen added a comment to T10936: arc feature error while creating a new feature branch.

This is almost definitely T8298, because % can't be properly escaped on Windows right now.

May 16 2016, 10:49 AM · Windows, Arcanist, Bug Report
rencris added a comment to T10936: arc feature error while creating a new feature branch.

This specific issue also happens on my computer, which is running Windows 7 SP1 with PHP 5.6.20

May 16 2016, 9:53 AM · Windows, Arcanist, Bug Report

May 15 2016

hach-que added a comment to T10969: Incorrect Phriction link in email.

It's not email client dependent - Phabricator is writing out the wrong URL when sending email.

May 15 2016, 5:26 AM · Phriction, Bug Report

May 14 2016

aklapper added a comment to T10969: Incorrect Phriction link in email.

Which email client is this about?

May 14 2016, 6:27 PM · Phriction, Bug Report
hach-que added a comment to T10969: Incorrect Phriction link in email.

Beat me to it :P

May 14 2016, 12:15 AM · Phriction, Bug Report

May 13 2016

joshuaspence created T10969: Incorrect Phriction link in email.
May 13 2016, 11:12 PM · Phriction, Bug Report
thoughtpolice moved T10635: Loading differential revision slow when lots of unit test messages exist from Backlog to Details on the Haskell.org board.
May 13 2016, 10:05 PM · Restricted Project, Haskell.org, Harbormaster, Performance
thoughtpolice added projects to T10635: Loading differential revision slow when lots of unit test messages exist: Harbormaster, Haskell.org.
May 13 2016, 10:03 PM · Restricted Project, Haskell.org, Harbormaster, Performance
thoughtpolice moved T10225: arc patch --nobranch leaves submodules in the wrong state from Backlog to Details on the Haskell.org board.
May 13 2016, 6:09 PM · Haskell.org, Restricted Project, Arcanist, Bug Report
thoughtpolice added a project to T10225: arc patch --nobranch leaves submodules in the wrong state: Haskell.org.
May 13 2016, 6:07 PM · Haskell.org, Restricted Project, Arcanist, Bug Report
eadler moved T10174: Adding a owner package to an audit when package auditing is disable leads to a "not applicable" commit review from Restricted Project Column to Restricted Project Column on the Restricted Project board.
May 13 2016, 6:02 PM · Restricted Project, Herald, Owners, Bug Report
patrick-ken created T10965: [Needs Audit] shows "Query Errors : This query specifies an invalid parameter.".
May 13 2016, 3:47 PM · Audit, Bug Report
epriestley closed T10963: libphutil/src/internationalization/PhutilTranslator.php:chooseVariant() hardcodes locales as Invalid.

This isn't a bug, the block has a performance-motivated explanation in a TODO comment above it:

May 13 2016, 12:06 PM · Bug Report
urzds created T10963: libphutil/src/internationalization/PhutilTranslator.php:chooseVariant() hardcodes locales.
May 13 2016, 11:27 AM · Bug Report

May 12 2016

epriestley closed T10958: Dead Link at Diviner as Resolved by committing Restricted Diffusion Commit.
May 12 2016, 3:30 PM · Documentation, Diviner, Bug Report
epriestley added a comment to T10958: Dead Link at Diviner.

Thanks!

May 12 2016, 3:20 PM · Documentation, Diviner, Bug Report
epriestley claimed T10958: Dead Link at Diviner.
May 12 2016, 3:20 PM · Documentation, Diviner, Bug Report
epriestley added a revision to T10958: Dead Link at Diviner: Restricted Differential Revision.
May 12 2016, 3:19 PM · Documentation, Diviner, Bug Report
Luke081515.2 updated the task description for T10952: Cannot edit pastes, if you want to keep the content.
May 12 2016, 2:31 PM · Paste, Bug Report
mrjoops added a comment to T10957: When logged in through an external auth provider, user is invited to re-create his account.

OK so they have to make it at step 3, before his first session expires, otherwise I'd have to update it directly in DB.

May 12 2016, 2:25 PM · Auth, Bug Report
epriestley closed T10955: Changing tags doesn't seem possible on repositories any more? as Resolved by committing Restricted Diffusion Commit.
May 12 2016, 2:17 PM · Diffusion, Bug Report
Luke081515.2 created T10958: Dead Link at Diviner.
May 12 2016, 2:16 PM · Documentation, Diviner, Bug Report
epriestley added a comment to T10957: When logged in through an external auth provider, user is invited to re-create his account.

They can link their accounts in Settings → External Accounts.

May 12 2016, 2:09 PM · Auth, Bug Report
mrjoops added a comment to T10957: When logged in through an external auth provider, user is invited to re-create his account.

OK for the security feature, but in my case LDAP is the only auth provider (username/password is disabled), so my users can't link their accounts.

May 12 2016, 2:08 PM · Auth, Bug Report
epriestley added a revision to T10955: Changing tags doesn't seem possible on repositories any more?: Restricted Differential Revision.
May 12 2016, 1:26 PM · Diffusion, Bug Report
epriestley added a comment to T10955: Changing tags doesn't seem possible on repositories any more?.

It's on an invisible page <_< >_>

May 12 2016, 1:17 PM · Diffusion, Bug Report
epriestley claimed T10955: Changing tags doesn't seem possible on repositories any more?.
May 12 2016, 1:17 PM · Diffusion, Bug Report
epriestley closed T10957: When logged in through an external auth provider, user is invited to re-create his account as Invalid.

This is expected. This is a security feature, not a bug. Just because you have a Phabricator user account named joe does not mean you have authority over the LDAP user account named joe, and vice versa. In many cases LDAP and Phabricator user account names differ. Users must prove they have control over external accounts by linking them before they can use those external accounts to access Phabricator.

May 12 2016, 1:16 PM · Auth, Bug Report
mrjoops added a comment to T10957: When logged in through an external auth provider, user is invited to re-create his account.

I didn't see anything related in recent changelogs, so I didn't update out instance.
But if you could wait a couple of days, I could try to reproduce.

May 12 2016, 12:27 PM · Auth, Bug Report
chad added a comment to T10957: When logged in through an external auth provider, user is invited to re-create his account.

Is the issue reproducible with an updated version of Phabricator?

May 12 2016, 12:16 PM · Auth, Bug Report
mrjoops created T10957: When logged in through an external auth provider, user is invited to re-create his account.
May 12 2016, 12:06 PM · Auth, Bug Report
hach-que created T10955: Changing tags doesn't seem possible on repositories any more?.
May 12 2016, 9:35 AM · Diffusion, Bug Report

May 11 2016

Luke081515.2 added a comment to T10952: Cannot edit pastes, if you want to keep the content.

Merci :)

May 11 2016, 10:56 PM · Paste, Bug Report
epriestley closed T10930: Exception in editengine for Paste as Resolved by committing Restricted Diffusion Commit.
May 11 2016, 10:35 PM · ApplicationEditor, Bug Report
epriestley closed T10952: Cannot edit pastes, if you want to keep the content as Resolved by committing Restricted Diffusion Commit.
May 11 2016, 10:35 PM · Paste, Bug Report
epriestley added a revision to T10930: Exception in editengine for Paste: Restricted Differential Revision.
May 11 2016, 10:34 PM · ApplicationEditor, Bug Report
epriestley added a revision to T10952: Cannot edit pastes, if you want to keep the content: Restricted Differential Revision.
May 11 2016, 10:34 PM · Paste, Bug Report
Luke081515.2 created T10952: Cannot edit pastes, if you want to keep the content.
May 11 2016, 9:26 PM · Paste, Bug Report
swisspol created T10951: Search shows document types corresponding to disabled apps.
May 11 2016, 4:02 PM · Contributor Onboarding, Search, Bug Report
epriestley closed T10949: Unable to apply patch 'phabricator:20160112.repo.02.uri.index.php': Table 'phabricator_repository.repository_uri' doesn't exist as Resolved.

Cool -- thanks for the report, and sorry about the issue! Some of the Diffusion/repository stuff has been a little choppy lately as we've rolled out changes in connection with T10922 / T10923. Let us know if you run into anything else.

May 11 2016, 2:11 PM · Bug Report
simevo added a comment to T10949: Unable to apply patch 'phabricator:20160112.repo.02.uri.index.php': Table 'phabricator_repository.repository_uri' doesn't exist.

Works like a charm. I never imagined this could be so quick ! Many thanks

May 11 2016, 2:09 PM · Bug Report
epriestley closed T10941: Subversion fails to execute pre-commit hook if CWD isn't readable as Resolved by committing Restricted Diffusion Commit.
May 11 2016, 1:48 PM · Diffusion, Subversion, Bug Report
epriestley closed T10815: Confusing diffusion error with empty, non-activated hosted repository as Resolved by committing Restricted Diffusion Commit.
May 11 2016, 1:39 PM · Diffusion, Bug Report
epriestley added a comment to T10949: Unable to apply patch 'phabricator:20160112.repo.02.uri.index.php': Table 'phabricator_repository.repository_uri' doesn't exist.

This should now be fixed in HEAD of master -- try updating again? You'll need to git pull in phabricator/ to pick up the change.

May 11 2016, 1:37 PM · Bug Report
epriestley added a comment to T10949: Unable to apply patch 'phabricator:20160112.repo.02.uri.index.php': Table 'phabricator_repository.repository_uri' doesn't exist.

This should be fixed shortly by D15876.

May 11 2016, 1:35 PM · Bug Report
simevo created T10949: Unable to apply patch 'phabricator:20160112.repo.02.uri.index.php': Table 'phabricator_repository.repository_uri' doesn't exist.
May 11 2016, 1:31 PM · Bug Report
clemvangelis added a comment to T10090: Updating Herald rule throws Unhandled Exception.

Indeed, it's fixed ! Thanks

May 11 2016, 7:07 AM · Herald, Bug Report

May 10 2016

eadler edited projects for T10933: You Shall Not Pass: Unknown Object (Repository), added: Diffusion; removed Differential.
May 10 2016, 6:30 PM · Diffusion, Bug Report
eadler added a project to T10933: You Shall Not Pass: Unknown Object (Repository): Differential.
May 10 2016, 6:30 PM · Diffusion, Bug Report
epriestley closed T10090: Updating Herald rule throws Unhandled Exception as Resolved.

This may have been fixed by D15507. Otherwise, we can't reproduce it, so we can't move forward. See Providing Reproduction Steps.

May 10 2016, 4:20 PM · Herald, Bug Report
maxie added a project to T10944: Subproject is not archived when parent project gets archived: Projects.
May 10 2016, 3:25 PM · Projects, Bug Report
maxie created T10944: Subproject is not archived when parent project gets archived.
May 10 2016, 3:24 PM · Projects, Bug Report
epriestley renamed T10941: Subversion fails to execute pre-commit hook if CWD isn't readable from Subversion commits fail with "Commit blocked by pre-commit hook (exit code 255) with no output" to Subversion fails to execute pre-commit hook if CWD isn't readable.
May 10 2016, 2:33 PM · Diffusion, Subversion, Bug Report
epriestley added a revision to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable: Restricted Differential Revision.
May 10 2016, 2:32 PM · Diffusion, Subversion, Bug Report
epriestley added a comment to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.

Awesome! Thanks for doing all the work to figure out what was going on. I'll get a diff out to apply that fix in a slightly nicer way.

May 10 2016, 2:30 PM · Diffusion, Subversion, Bug Report
dpotter added a comment to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.

Yes, that corrects the issue!

May 10 2016, 2:29 PM · Diffusion, Subversion, Bug Report
epriestley added a revision to T10815: Confusing diffusion error with empty, non-activated hosted repository: Restricted Differential Revision.
May 10 2016, 2:15 PM · Diffusion, Bug Report
epriestley added a comment to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.

Thanks for hunting that down!

May 10 2016, 12:27 PM · Diffusion, Subversion, Bug Report
epriestley closed T10406: Archived repositories aren't imported as Resolved by committing Restricted Diffusion Commit.
May 10 2016, 12:15 PM · Bug Report, Diffusion
dpotter added a comment to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.

OK, I found the issue and a fix/workaround:

May 10 2016, 3:45 AM · Diffusion, Subversion, Bug Report

May 9 2016

epriestley added a comment to T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.

I can't reproduce this:

May 9 2016, 11:50 PM · Diffusion, Subversion, Bug Report
epriestley merged task T10942: Subversion repositories are created with incorrect default URI into T10923: Diffusion API/URI Errata.
May 9 2016, 11:42 PM · Bug Report
epriestley added a comment to T10406: Archived repositories aren't imported.

D15873 makes this explicit in the dialog.

May 9 2016, 11:25 PM · Bug Report, Diffusion
epriestley added a revision to T10406: Archived repositories aren't imported: Restricted Differential Revision.
May 9 2016, 11:21 PM · Bug Report, Diffusion
dpotter updated the task description for T10942: Subversion repositories are created with incorrect default URI.
May 9 2016, 9:56 PM · Bug Report
dpotter updated the task description for T10942: Subversion repositories are created with incorrect default URI.
May 9 2016, 9:49 PM · Bug Report
dpotter updated the task description for T10942: Subversion repositories are created with incorrect default URI.
May 9 2016, 9:48 PM · Bug Report
dpotter created T10942: Subversion repositories are created with incorrect default URI.
May 9 2016, 9:46 PM · Bug Report
dpotter created T10941: Subversion fails to execute pre-commit hook if CWD isn't readable.
May 9 2016, 9:30 PM · Diffusion, Subversion, Bug Report
OCram created T10936: arc feature error while creating a new feature branch.
May 9 2016, 9:57 AM · Windows, Arcanist, Bug Report
epriestley added a comment to T10934: Phabricator not working with HHVM + proxygen server mode.

Or see @avivey's far more complete response.

May 9 2016, 4:44 AM · Bug Report
epriestley closed T10934: Phabricator not working with HHVM + proxygen server mode as Wontfix.

We don't support HHVM.

May 9 2016, 4:43 AM · Bug Report
avivey closed T10934: Phabricator not working with HHVM + proxygen server mode as Invalid.
May 9 2016, 4:43 AM · Bug Report
netroby created T10934: Phabricator not working with HHVM + proxygen server mode.
May 9 2016, 3:43 AM · Bug Report

May 8 2016

epriestley added a comment to T10933: You Shall Not Pass: Unknown Object (Repository).

The difference with our interface : I have not installed "http git" only ssh git

May 8 2016, 9:16 PM · Diffusion, Bug Report
karima.rafes added a comment to T10933: You Shall Not Pass: Unknown Object (Repository).

The difference with our interface : I have not installed "http git" only ssh git

May 8 2016, 9:13 PM · Diffusion, Bug Report
epriestley added a comment to T10933: You Shall Not Pass: Unknown Object (Repository).

I cannot change the Space by default

May 8 2016, 9:01 PM · Diffusion, Bug Report
epriestley added a comment to T10933: You Shall Not Pass: Unknown Object (Repository).

I can't reproduce this. Here are the repository policies:

May 8 2016, 9:00 PM · Diffusion, Bug Report
karima.rafes created T10933: You Shall Not Pass: Unknown Object (Repository).
May 8 2016, 7:55 PM · Diffusion, Bug Report
chad added a comment to T10932: Daily view of calendar uses 12h (am/pm) marking of time, although preferences set to 24h.

See https://secure.phabricator.com/book/phabricator/article/prototypes/

May 8 2016, 1:30 PM · Calendar, Bug Report
• SirRichardVanEsquire added a comment to T10356: Unhandled Exception ("FilesystemException") after adding Github repo.

I ran into this same issue, it was caused by user error, but the error handling of diffusion could still be fixed. Here's how to reproduce:

May 8 2016, 12:08 AM · Diffusion, Bug Report

May 7 2016

Danny_B created T10932: Daily view of calendar uses 12h (am/pm) marking of time, although preferences set to 24h.
May 7 2016, 10:20 PM · Calendar, Bug Report

May 6 2016

hut added a comment to T10931: Changing mysql.implementation setting doesn't work.

That actually works! Thanks :)

May 6 2016, 10:58 PM · Setup, Database, Bug Report
epriestley added a comment to T10931: Changing mysql.implementation setting doesn't work.

Haha, alright. You might be able to add it to conf/local/local.json manually as a workaround, but I'm not sure if that'll work or not.

May 6 2016, 10:40 PM · Setup, Database, Bug Report
hut added a comment to T10931: Changing mysql.implementation setting doesn't work.

It involves dirty solutions to idiosyncratic problems, maybe we're both better off not talking about it.

May 6 2016, 10:36 PM · Setup, Database, Bug Report