profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/marxin/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

marxin/cvise 82

Super-parallel Python port of the C-Reduce

marxin/AdventOfCodeStats 8

Statistics about Advent Of Code

marxin/bash-completion 1

Programmable completion functions for bash

marxin/AoC2016 0

Advent of Code 2016

marxin/AoC2018 0

Advent of Code 2018

marxin/AoC2019 0

Advent of Code 2019

marxin/AoC2020 0

Advent of Code 2020

marxin/asmdiff 0

Tool for comparing assembler files/dumps

issue commentmarxin/cvise

Cvising "PASS_BUG_INFO"

And you can also lower GIVEUP_CONSTANT = 50000 in cvise/utils/testing.py and then grep for the message.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Cvising "PASS_BUG_INFO"

Sorry. Well, it's not so easy, but you can call cvise --start-with-pass IncludesPass and then somehow identify with --verbose that it's not doing progress. Do you have a reproducer I can take a look at?

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Infinitely expanding file with `remove-nested-function`

All right, so the input has grown 10x during the ClangPass::remove-nested-function pass. I'm going to add a bail-out limit when pass grows an input rapidly.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Infinitely expanding file with `remove-nested-function`

Hmm, this one is quite ugly. Btw. is the original test-case an infinite loop? I can imagine a pass termination when it rapidly grows the input.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Cvising "PASS_BUG_INFO"

Thank you very much for the testing effort, I would expect quite some cvise crashes for various inputs. Luckily, one is not blocked by a single crash during a reduction, and the reduction typically finishes.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Segmentation violation with multiple passes

What an input test case, please report it to the upstream creduce project. Hope they can help us with that.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Assertion failure with `simple-inliner`

Thanks for the bug. Can you please report it to upstream creduce project, they should know more.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Assertion failure with `param-to-local`

Thanks for the bug. Can you please report it to upstream creduce project, they should know more.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Assertion failure with `lift-assignment-expr`

Thanks for the bug. Can you please report it to upstream creduce project, they should know more.

andrewvaughanj

comment created time in 2 days

issue commentmarxin/cvise

Assertion failure with `remove-namespace`

Thanks for the bug. Can you please report it to upstream creduce project, they should know more.

andrewvaughanj

comment created time in 2 days

issue commentcsmith-project/creduce

Please make a release

As an alternative, I can offer https://github.com/marxin/cvise, which supports all LLVM releases since version 9.

0-wiz-0

comment created time in 2 days

push eventrpm-software-management/rpmlint

Klaus Kämpf

commit sha e8fc15b0e6d6fc2255c4bd074a46898818000532

Add 'opensearch' OpenSearch (https://github.com/opensearch-project/OpenSearch) is an open-source fork of Elasticsearch

view details

Martin Liska

commit sha 86f10b004622ae7d88ed3393d984d8b4ae90f1ed

Merge branch 'main' into opensuse

view details

push time in 3 days

delete branch marxin/rpmlint

delete branch : mini-mode

delete time in 3 days

push eventrpm-software-management/rpmlint

Martin Liska

commit sha 46f9d302c8d4a24223098ad1dc82c5bf095b657f

Come up with --mini-mode option. The option will be used by rpmlint-lint package wrapper.

view details

marxin

commit sha e96614fa1e344da4a09b7a6473bc86bcc74dcd4d

Merge pull request #678 from marxin/mini-mode Come up with --mini-mode option.

view details

push time in 3 days

PR merged rpm-software-management/rpmlint

Come up with --mini-mode option.

The option will be used by rpmlint-lint package wrapper.

+14 -4

13 comments

6 changed files

marxin

pr closed time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

Thanks. Anyway, I would appreciate it if you would discuss the mandatory use of rpmlint in Factory with Dimstar.

marxin

comment created time in 3 days

push eventmarxin/binutils-cve

Martin Liska

commit sha abb25e62db59c1bf1a1620c1cad221ca981dedb2

Update to 2.37.

view details

push time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

Ah, that's it :/ So are fine for now with this pull request, that's purely opensuse-specific?

marxin

comment created time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

I don't think I can comment in that ticket, as I'm not a SUSE employee. I didn't get a Jira notification about it.

Hm, I found your account and added you to https://jira.suse.com/browse/OBS-88. Your username seems to be Pharaoh_Atem, am I right?

marxin

comment created time in 3 days

issue commentrpm-software-management/rpmlint

Warn against python setup.py test construct

I see. So should we warn about any %python_exec setup.py test call in a spec file? Are you willing to create a pull request for it? Looking at openSUSE:Factory package, there are quite some:

grep 'setup.py test' *.spec | wc -l
162
mcepl

comment created time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

As an rpmlint upstream maintainer and an openSUSE person, I would strongly advise against it. When OBS does bootstrap cycles, it goes through and builds them and then rebuilds them again in the main project itself. Because of that, there's no particular reason to make the bootstrap phase include rpmlint. Since hacksaw stuff is already happening there anyway, it's not particularly useful.

I fully got that and support it! I've just added you to a Jira ticket where the problem was discussed. Feel free to express the need to make rpmlint only an optional dependency.

marxin

comment created time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

That doesn't solve the problem for you. You'd still need the Python interpreter because virtualenvs do not include the interpreter.

We do have it, but does not work due to: Because it runs rpm, which expands every macro in the spec file, which includes things like %pythons and so on, and so it ends up running python3.9 which then complains bitterly since most of it is removed.

Moreover, I don't know why you need an extra wrapper or mode when you could just make those those things weak dependencies instead.

Right now, the spellchecker is a weak dependency but generates (none): W: unable to init enchant, spellchecking disabled..

The -mini convention isn't really for minimized builds, it's for producing hacksaw versions for the OBS build environment.

Yes, we are aware of the hackish approach.

(Personally, I think any package that is called -mini in openSUSE needs a better name, because it's clearly wrong.).

It also doesn't actually make sense to have rpmlint as a bootstrap package. It's not required for building packages, and trying to force it in that early is just asking for pain.

Well, that's decision made by openSUSE:Factory people and they want to have rpmlint 2.0 as a mandatory requirement for now.

marxin

comment created time in 3 days

issue commentrpm-software-management/rpmlint

license parsing corner case

Note that rpmlint uses SPDX License List as defined here: https://spdx.org/licenses/. So my guess is that you need something like GPL-2.0-with-autoconf-exception, in your case sip6..

spotrh

comment created time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

Instead of python39, we create a virtualenv.

marxin

comment created time in 3 days

pull request commentrpm-software-management/rpmlint

Come up with --mini-mode option.

In order to keep a limited number of bootstrap cycle packages, we don't want to depend on pyenchant and python39. The pull request goes only to opensuse branch..

marxin

comment created time in 3 days

PR opened rpm-software-management/rpmlint

Come up with --mini-mode option.

The option will be used by rpmlint-lint package wrapper.

+14 -4

0 comment

6 changed files

pr created time in 3 days

push eventrpm-software-management/rpmlint

Martin Liska

commit sha 18b999a01e14485b1735a612d826b05825118184

Add missing parenthesis.

view details

Martin Liska

commit sha 7f3b4d84b979690604594fbf116dbe573d6da2e9

Merge branch 'main' into opensuse

view details

push time in 3 days

create barnchmarxin/rpmlint

branch : mini-mode

created branch time in 3 days

push eventrpm-software-management/rpmlint

Martin Liska

commit sha 18b999a01e14485b1735a612d826b05825118184

Add missing parenthesis.

view details

push time in 3 days

push eventmarxin/texi2rst-generated

marxin

commit sha 7e2f616b4ad280e9b6d768e05f1153d29fcff439

Built (push) davidmalcolm/texi2rst@de59762f5800ee49918b8210364718a4ccab828e

view details

marxin

commit sha ede6330160d294e03621dbb545452324116482c3

Merge branch 'raw'

view details

push time in 13 days