Wsparcie dla formatu uBlacklist #5

Merged
wiktor merged 10 commits from ublacklist-format into master 2025-05-29 15:51:53 +02:00
Collaborator
No description provided.
wiktor added 7 commits 2025-05-25 21:59:18 +02:00
Collaborator

Sprawdziłem.

  1. make build nie działa. Kończy się errorem: $ make build \ make: *** Brak reguł do wykonania obiektu 'build'. Stop.
  2. Polecenie make działa, ale buduje tylko adguard.txt. Brakuje budowy pliku ublacklist.txt
  3. Czy podział na satire, mirror, reflink, seo był dyskutowany wcześniej?
Sprawdziłem. 1. `make build` nie działa. Kończy się errorem: `$ make build \ make: *** Brak reguł do wykonania obiektu 'build'. Stop.` 2. Polecenie `make` działa, ale buduje tylko `adguard.txt`. Brakuje budowy pliku `ublacklist.txt` 3. Czy podział na satire, mirror, reflink, seo był dyskutowany wcześniej?
wiktor added 2 commits 2025-05-27 17:32:38 +02:00
Author
Collaborator
  1. make build nie działa. Kończy się errorem: $ make build \ make: *** Brak reguł do wykonania obiektu 'build'. Stop.
  2. Polecenie make działa, ale buduje tylko adguard.txt. Brakuje budowy pliku ublacklist.txt

Faktycznie. Zrefaktoryzowałem Makefile. Już powinno działać.

Czy podział na satire, mirror, reflink, seo był dyskutowany wcześniej?

Tak. Konsultowałem to z @kuba w #1

> 1. `make build` nie działa. Kończy się errorem: $ make build \ make: *** Brak reguł do wykonania obiektu 'build'. Stop. > 2. Polecenie `make` działa, ale buduje tylko adguard.txt. Brakuje budowy pliku `ublacklist.txt` Faktycznie. Zrefaktoryzowałem Makefile. Już powinno działać. > Czy podział na satire, mirror, reflink, seo był dyskutowany wcześniej? Tak. Konsultowałem to z @kuba w https://git.internet-czas-dzialac.pl/icd/blocklisty/pulls/1
Collaborator

Faktycznie. Zrefaktoryzowałem Makefile. Już powinno działać.

Sprawdziłem. Jak weźmie się sklonuje repozytorium oraz doda nową stronę, to budowanie nie działa:

$ make build
make: Nie ma nic do zrobienia w 'build'.

Działa dopiero jak najpierw dam make clean a później make build. W poprzedniej wersji komenda make dopisywała nowe rzeczy do listy, wstępne make clean nie było potrzebne.

> Faktycznie. Zrefaktoryzowałem Makefile. Już powinno działać. Sprawdziłem. Jak weźmie się sklonuje repozytorium oraz doda nową stronę, to budowanie nie działa: ```bash $ make build make: Nie ma nic do zrobienia w 'build'. ``` Działa dopiero jak najpierw dam `make clean` a później `make build`. W poprzedniej wersji komenda `make` dopisywała nowe rzeczy do listy, wstępne `make clean` nie było potrzebne.
Collaborator

Może coś takiego?

CONVERTER = python3 src/converter.py
BLOCKLISTS = seo-nonsense reflink-spam satire mirror
TARGET_FORMATS = adguard ublacklist

TARGETS = $(foreach blocklist, $(BLOCKLISTS), $(foreach targetformat, $(TARGET_FORMATS), dist/$(blocklist)/$(targetformat).txt))

build: $(TARGETS)

dist/%/adguard.txt: src/%-source.json
	mkdir -p dist/$*
	$(CONVERTER) --inputfile src/$*-source.json --targetformat adguard --outputfile $@

dist/%/ublacklist.txt: src/%-source.json
	mkdir -p dist/$*
	$(CONVERTER) --inputfile src/$*-source.json --targetformat ublacklist --outputfile $@

clean:
	rm -rf dist/*

all: build

.PHONY: build clean all

Chciałem pushnąć to na branch, ale nie mam potrzebnych uprawnień

Może coś takiego? ```make CONVERTER = python3 src/converter.py BLOCKLISTS = seo-nonsense reflink-spam satire mirror TARGET_FORMATS = adguard ublacklist TARGETS = $(foreach blocklist, $(BLOCKLISTS), $(foreach targetformat, $(TARGET_FORMATS), dist/$(blocklist)/$(targetformat).txt)) build: $(TARGETS) dist/%/adguard.txt: src/%-source.json mkdir -p dist/$* $(CONVERTER) --inputfile src/$*-source.json --targetformat adguard --outputfile $@ dist/%/ublacklist.txt: src/%-source.json mkdir -p dist/$* $(CONVERTER) --inputfile src/$*-source.json --targetformat ublacklist --outputfile $@ clean: rm -rf dist/* all: build .PHONY: build clean all ``` Chciałem pushnąć to na branch, ale nie mam potrzebnych uprawnień
wiktor added 1 commit 2025-05-28 22:12:32 +02:00
Co-authored-by: cichy1173 <cichy1173@protonmail.com>
Author
Collaborator
$ make
make: Nothing to be done for 'build'
$ make clean
rm -rf dist/*
$ make
mkdir -p dist/seo-nonsense
python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat adguard --outputfile dist/seo-nonsense/adguard.txt
[INFO] Generated in 7 ms
mkdir -p dist/seo-nonsense
python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat ublacklist --outputfile dist/seo-nonsense/ublacklist.txt
[INFO] Generated in 4 ms
mkdir -p dist/reflink-spam
python3 src/converter.py --inputfile src/reflink-spam-source.json --targetformat adguard --outputfile dist/reflink-spam/adguard.txt
[INFO] Generated in 4 ms
mkdir -p dist/reflink-spam
python3 src/converter.py --inputfile src/reflink-spam-source.json --targetformat ublacklist --outputfile dist/reflink-spam/ublacklist.txt
[INFO] Generated in 4 ms
mkdir -p dist/satire
python3 src/converter.py --inputfile src/satire-source.json --targetformat adguard --outputfile dist/satire/adguard.txt
[INFO] Generated in 4 ms
mkdir -p dist/satire
python3 src/converter.py --inputfile src/satire-source.json --targetformat ublacklist --outputfile dist/satire/ublacklist.txt
[INFO] Generated in 4 ms
mkdir -p dist/mirror
python3 src/converter.py --inputfile src/mirror-source.json --targetformat adguard --outputfile dist/mirror/adguard.txt
[INFO] Generated in 4 ms
mkdir -p dist/mirror
python3 src/converter.py --inputfile src/mirror-source.json --targetformat ublacklist --outputfile dist/mirror/ublacklist.txt
[INFO] Generated in 4 ms

dodanie nowego wpisu do seo-nonsense-source.json

$ make
mkdir -p dist/seo-nonsense
python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat adguard --outputfile dist/seo-nonsense/adguard.txt
[INFO] Generated in 4 ms
mkdir -p dist/seo-nonsense
python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat ublacklist --outputfile dist/seo-nonsense/ublacklist.txt
[INFO] Generated in 4 ms

Wydaje się działać jak należy 😄

``` $ make make: Nothing to be done for 'build' ``` ``` $ make clean rm -rf dist/* ``` ``` $ make mkdir -p dist/seo-nonsense python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat adguard --outputfile dist/seo-nonsense/adguard.txt [INFO] Generated in 7 ms mkdir -p dist/seo-nonsense python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat ublacklist --outputfile dist/seo-nonsense/ublacklist.txt [INFO] Generated in 4 ms mkdir -p dist/reflink-spam python3 src/converter.py --inputfile src/reflink-spam-source.json --targetformat adguard --outputfile dist/reflink-spam/adguard.txt [INFO] Generated in 4 ms mkdir -p dist/reflink-spam python3 src/converter.py --inputfile src/reflink-spam-source.json --targetformat ublacklist --outputfile dist/reflink-spam/ublacklist.txt [INFO] Generated in 4 ms mkdir -p dist/satire python3 src/converter.py --inputfile src/satire-source.json --targetformat adguard --outputfile dist/satire/adguard.txt [INFO] Generated in 4 ms mkdir -p dist/satire python3 src/converter.py --inputfile src/satire-source.json --targetformat ublacklist --outputfile dist/satire/ublacklist.txt [INFO] Generated in 4 ms mkdir -p dist/mirror python3 src/converter.py --inputfile src/mirror-source.json --targetformat adguard --outputfile dist/mirror/adguard.txt [INFO] Generated in 4 ms mkdir -p dist/mirror python3 src/converter.py --inputfile src/mirror-source.json --targetformat ublacklist --outputfile dist/mirror/ublacklist.txt [INFO] Generated in 4 ms ``` *dodanie nowego wpisu do `seo-nonsense-source.json`* ``` $ make mkdir -p dist/seo-nonsense python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat adguard --outputfile dist/seo-nonsense/adguard.txt [INFO] Generated in 4 ms mkdir -p dist/seo-nonsense python3 src/converter.py --inputfile src/seo-nonsense-source.json --targetformat ublacklist --outputfile dist/seo-nonsense/ublacklist.txt [INFO] Generated in 4 ms ``` Wydaje się działać jak należy 😄
Collaborator

Jako iż nie mam żadnych uprawnień w tym repozytorium, to nie dam Ci checka approve, ale masz ode mnie approve. Zmergować pr może @kuba

Jako iż nie mam żadnych uprawnień w tym repozytorium, to nie dam Ci checka `approve`, ale masz ode mnie **approve**. Zmergować pr może @kuba
wiktor merged commit 11a1c14155 into master 2025-05-29 15:51:53 +02:00
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: icd/blocklisty#5
No description provided.