Page MenuHomePhabricator

kugel- (Thomas Martitz)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Feb 10 2016, 7:31 AM (148 w, 6 d)
Availability
Available

Recent Activity

Jan 9 2018

kugel- added a comment to rARCf1ba27ffedd5: Add "arc diff --draft".

Is this going to land in master any time soon? Our company would love to use the draft feature.

Jan 9 2018, 8:33 PM

Nov 30 2017

kugel- added a comment to T4333: `arc land` should provide a mechanism to land revisions by other authors while preserving authorship meta-data.

Also, in case of "arc land --merge", the merge commit is falsely authored by the revision author.

Nov 30 2017, 3:41 PM · Arcanist
kugel- added a comment to T4333: `arc land` should provide a mechanism to land revisions by other authors while preserving authorship meta-data.

arc land still prints a warning in this scenario. Is there a way to disable that through .arcconfig? arc land itself doesn't seem to have an option to skip the warning.

Nov 30 2017, 3:37 PM · Arcanist

Aug 5 2017

kugel- added a comment to T12952: Reviewer not able to see diff.

We require the "a revision's reviewers can always..." behavior. Please implement it instead of changing the text.

Aug 5 2017, 8:54 PM · Differential, Bug Report

Feb 3 2017

kugel- updated subscribers of T10608: Arc diff failing to generate diff with svn 1.9.3.

@epriestley can you have a look at this, please?

Feb 3 2017, 9:56 AM · Arcanist, Bug Report
kugel- added a comment to T12190: In PHP7, include-time warnings from source files are ignored.

@epriestley can you have a look at this, please?

Feb 3 2017, 9:55 AM · Infrastructure

Jan 17 2017

kugel- added a comment to D17212: Explicitly warn the user multiple times when they try to register an external account with an existing email.

Just for the record, this change works for me. Many thanks for quickly resolving my problem!

Jan 17 2017, 8:25 PM
kugel- added a comment to T12113: Email in LDAP registration form is empty.

Alright, then this is it. I think a visual indication would be helpful. But apart from that, this can be closed.

Jan 17 2017, 6:53 PM · Bug Report (Needs Information)
kugel- added a comment to T12113: Email in LDAP registration form is empty.

So I asked a coworker to register with *his* credentials, and it turns out he can't see the email field. I strongly suspect it has to do with the fact that I already registered with the same email address when setting up the admin account initially.

Jan 17 2017, 8:39 AM · Bug Report (Needs Information)
kugel- added a comment to T12113: Email in LDAP registration form is empty.

Here is my ldap auth config

Jan 17 2017, 8:20 AM · Bug Report (Needs Information)
kugel- added a comment to T12113: Email in LDAP registration form is empty.

I tried again on a clean phabricator install (git master as of 10 minutes ago) on Ubuntu 16.04 and php7.1 from https://launchpad.net/~ondrej/+archive/ubuntu/php and still observe the same issue. So it looks like it's specific to our ldap server? Does the output I gave you above ring any bell?

Jan 17 2017, 8:10 AM · Bug Report (Needs Information)
kugel- added a comment to T12113: Email in LDAP registration form is empty.

Thank you for your extremely intensive efforts to reproduce my issue.

Jan 17 2017, 7:35 AM · Bug Report (Needs Information)

Jan 16 2017

kugel- created T12113: Email in LDAP registration form is empty.
Jan 16 2017, 2:45 PM · Bug Report (Needs Information)
kugel- added a comment to D14244: Added the ability to configure the email attribute for LDAP.

I tested this. Perhaps I'm missing something, but with this patch, when a new user registers using ldap, the email address field is NOT populated. I thought this was the purpose of this change?

Jan 16 2017, 1:48 PM

Jun 27 2016

kugel- added a comment to T10608: Arc diff failing to generate diff with svn 1.9.3.

I am also affected. Is there any known workaround?

Jun 27 2016, 1:06 PM · Arcanist, Bug Report

Jun 9 2016

kugel- added a comment to T11120: HTML mails malformed.

How can I enable base64 encoding within phabricator?

Jun 9 2016, 8:19 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

The client was thunderbird and the webclient (based Open-Xchange). I received it on my personal mail account, outside the company.

Jun 9 2016, 7:57 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

Here's the mail that I received for your comment. You can see that the M stream is broken with "!" in two places. This seems to match the behavior I observed initially, although there was also an extraneous blank.

Jun 9 2016, 7:33 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

The webmail client allows me to view the raw mail. It's malformed as well.

Jun 9 2016, 1:00 PM · Mail, Bug Report
kugel- added a comment to T11120: HTML mails malformed.

bin/mail show-outbound --id <id> --dump-html shows the mail fine.

Jun 9 2016, 12:56 PM · Mail, Bug Report
kugel- added a revision to T11120: HTML mails malformed: D15885: Send HTML email by default.
Jun 9 2016, 12:46 PM · Mail, Bug Report
kugel- added a task to D15885: Send HTML email by default: T11120: HTML mails malformed.
Jun 9 2016, 12:46 PM
kugel- created T11120: HTML mails malformed.
Jun 9 2016, 12:42 PM · Mail, Bug Report

Mar 7 2016

kugel- added a comment to D9922: Fix `arc` issues with non-English systems.

@epriestley this change works for us in de_DE locales. It would be nice if this was merged.

Mar 7 2016, 1:55 PM

Feb 10 2016

kugel- added a comment to T6635: The `arc which` command triggers notices.

@epriestley I also get this error. What about your fix?

Feb 10 2016, 1:24 PM · Subversion, Arcanist
kugel- created T10313: Maintain authorship for SVN repositories.
Feb 10 2016, 8:15 AM · Restricted Project, Arcanist, Feature Request
kugel- added a comment to T10312: Always interpret the first line of commit messages as a title, even if it begins with a recognized label like "Test Plan:".

I encountered this on a svn repository, but what do you do for git repositoires where "label: one line summary" is considered best practice for commit messages?

Feb 10 2016, 8:10 AM · Restricted Project, Diffusion, Differential, Arcanist, Bug Report
kugel- created T10312: Always interpret the first line of commit messages as a title, even if it begins with a recognized label like "Test Plan:".
Feb 10 2016, 7:35 AM · Restricted Project, Diffusion, Differential, Arcanist, Bug Report