GitLab

Fixes: c626d5ea ("Add robots.txt")
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Disallow all crawling.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

When running "git pw apply --revision N SERIES_ID" it always says
that it applies the newest revision even though it does the correct
thing - it applies the requested one.

This patch fixes the logic to print the correct revision.

This fixes #22Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Let's address a common gotcha - clicking on a project name in the navbar
used to navigate us to the main page (i.e. `/`), since this patch it
will get us to the series list of the current project instead.

This fixes #30.
Signed-off-by: Manish Kumar <manish3.kumar@intel.com>
Reviewed-by: Martin Peres <martin.peres@linux.intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

The all-new path() does not take regexps.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Wrong view, wrong way of passing view arguments.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

There were not used anyway.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

use_for_related_fields is gone in newer Django

We don't seem to use the manager on related fields anyway, so we are
good to remove it.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Makes sense only with Python 2 which we do not support.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

We have already removed Python 2 support, let's remove the leftovers
too.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Removed with never versions of Django.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Changes with never version.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Deprecated since forever, gone with newer Django.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Middleware is now defined slightly differently and we dropped _CLASSES
from the used middleware list in setting.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Changed in newer versions of Django. Deprecated since forever.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

2 months 10 days 6 hours and 41 minutes left. Time to move on.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Like on the project's patches list we can present tag counts (acked-by,
tested-by, etc.) on the series' patches lists too.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Debian Buster now has up-to-date nodejs so we don't need to pull it from
a third party repository.

Sadly npm generates warnings on stderr as it is not the latest one that
should come bundled with node but a Debian-maintained one, which is
quite old compared to the node version. The warnings are harmless
though.

Bugzilla: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=904422Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Celery 4.3 is compatible with Python 3.7
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Line was too long.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Cover letters are now using details/summary HTML block to be expandable
with the latest one being expanded by default.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Some servers have tendency to send non-RFC compliant headers which
causes patchwork to hiccup. Their admins are reluctant to act on it and
we cannot just force whole bunch of users to switch providers.

Let's do our bit of the robustness principle and be more liberal about
what we take.

There are actually two solution in place, one for PY2 and one for PY3.

PY2 is fine with reading the malformed file, but fails at header
encoding, so we just need to catch the exception and discard the header.

PY3 is a bit more complex. Since there were a lot of changes in how
files and strings are handled, we have to read the mail explicitly as
binary otherwise we get decode errors, hence the sys.stdin.buffer.

The changes in the email module and the addition of
message_from_binary_file() takes care of the issue for us.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Let's say that patch was added in 2nd revision of series. The following
URL can be used to browse the patch within that context:

/patch/123/?series=12&rev=2

If someone by mistake changes the revision to 1, which does not contain
patch 123 it used to result in error 500.

Let's be more graceful and 404 with some explanation instead.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

When creating a bundle or changing patch status a POST request is made.
This caused the whole section that handled navigation to be omitted,
which resulted in missing navigation links.

Let's drop the method checks there.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Instead 'mock' it with new instance of the class, so the results come
back empty.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

One extra "s" in "messages", which was causing the following exception
when bulk updating patches with no patches selected:

  File "/srv/patchwork.freedesktop.org/lib/apache2/../../patchwork/views/__init__.py" in process_multiplepatch_form
    157.         context['messsages'] += ["No patches selected; nothing updated"]

  Exception Type: KeyError
  Exception Value: 'messsages'
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

If navigating to a patch from a series/revision view a ?series=x&rev=y
are appended to the URL.

This is later used on the patch view to determine whether browsing this
patch from the series view. If that's the case new navigation options to
go to next and previous patches in series are available.

Closes #13Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Complementary to /mbox/ and /mbox_with_cover/.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>